-
Notifications
You must be signed in to change notification settings - Fork 273
Streamline Makefiles #657
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
Comments
See #497 - I've been gradually introducing this approach as I modify make files anyway. |
That's a good approach. We need to communicate further, as not everyone is aware of these guidelines (e.g. 055f515). |
On Mon, 2017-03-20 at 06:58 -0700, Joel Allred wrote:
That's a good approach. We need to communicate further, as not everyone is aware of these guidelines (e.g. 055f515).
Or else completely rewrite the Makefiles at once.
Perhaps it could be added to the lint script so that it continues to be
enforced.
|
It could. Who's in charge of that? |
So in waiting for sprint planning I have finally written said script to sort out make files and updated the linked PR, thanks for nudging me to do this :) |
There should be one filename per line in Makefiles.
Each line should end with an
\
, even the last one.A list of lines should thus end with a comment such as:
# Empty last line
.This will make merging of branches much easier.
The text was updated successfully, but these errors were encountered: