Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

Issue #2536: Makefile Standardization#19

Open
lrowang wants to merge 3 commits into
masterfrom
2536-standardize-makefile
Open

Issue #2536: Makefile Standardization#19
lrowang wants to merge 3 commits into
masterfrom
2536-standardize-makefile

Conversation

@lrowang

@lrowang lrowang commented Jan 17, 2017

Copy link
Copy Markdown

Hi @kostajh -

Proposed Makefile changes as part of Savas's ongoing internal project standardization!

  • Add make help script
  • Organized and edit targets to match new standards
  • Update README

This one is pretty straightforward, but let me know if you want to discuss anything!

Issue #2533: Add error field data for all fields
@kostajh

kostajh commented Mar 6, 2017

Copy link
Copy Markdown
Contributor

@lrowang do you mind rebasing this when you have the chance?

lrowang added 2 commits March 9, 2017 20:14
Add help script that displays summary of commonly used targets.
Reorganize and edit targets per the new internal standards.
Specify the new `make help` script.
@lrowang lrowang force-pushed the 2536-standardize-makefile branch from e9cbff7 to 63687e6 Compare March 10, 2017 04:17
@lrowang

lrowang commented Mar 10, 2017

Copy link
Copy Markdown
Author

Hi @kostajh - Rebased! Phew it felt dicey -- good practice for me! I was careful to compare the actual steps within each target command and they should match what's on master currently -- so my edits should just be adding comments and reordering for the make help standardization.

Let me know if there are any issues!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants