-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor additions to the software framework #26
Conversation
Update README with how to lint ignore lines Add resultant binary to .gitignore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the empty temp
file? Do we need it?
Sorry, we do not. I only used it for testing pre-commit hook. I needing some change to make a commit. Will remove the temp file and recommit shortly. |
Can you also add a note to the README regarding the linter not being 100% accurate and style guide having preference? |
No, I would prefer first changing the linter to reflect the style guide. Do not approve yet then, we shall resolve the inconsistencies together. |
There is now a separate target to allow for `make clean`ing of linter's stuff only so that you can re-lint without recompiling. Also, `make` now first compiles the project and then lints. It is more important to have code that compiles than code that lints (although the latter is still super important).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Any comments @mkristien? |
looks good to me |
Update README with how to lint ignore lines
Add resultant binary to .gitignore
Add compilation flags to Makefile