-
Notifications
You must be signed in to change notification settings - Fork 38.7k
doc: Add developer notes about gitignore #6878
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
Conversation
|
Good. You should mention the simple command line however which does most of the hard work as per https://help.github.com/articles/ignoring-files/#create-a-global-gitignore |
|
Thanks for the information, very helpful! |
|
ACK |
|
ACK. |
- Add developer notes about `.gitignore.` - Remove qt creator specific files from gitignore, to be consistent.
cf45f8b to
dca7bd3
Compare
Added a mention of that - personally I prefer editing the config file manually, instead of memorizing the command, don't think it's very hard work :) |
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.
Also fairly standard practice in open-source environments... (Not saying we have to do it, but this description isn't correct.)
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.
In my experience in - much more heterogeneous, multiplatform - open source projects it's rare to see ignores for specific tools that some of the developers use
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.
I think the point is that ignore files should always be specific to the project, and not to the developer's environment. So IDE files for example should always be in the user's own gitignore and not the project source's.
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.
I'm thinking things like *~
|
ACK |
|
ACK Meh, "doesn't hurt" on the dev notes. |
dca7bd3 doc: Add developer notes about gitignore (Wladimir J. van der Laan)
.gitignore.Context: discussion in #6813, this is a bit of a FAQ so worth mentioning.