search

pez2001 / razer_chroma_drivers

A collection of Linux drivers for Razer Chroma devices, it supports all lighting modes and includes a daemon for advanced effects + gui configuration app
GNU General Public License v2.0
231 stars 30 forks source link

git-ignore compiled products #92

Closed jeffkowalski closed 8 years ago

jeffkowalski commented 8 years ago

Ignore the .cmd files, and all binaries produced during compilation

terricain commented 8 years ago

Is it worth it?, What's wrong with make clean?

jeffkowalski commented 8 years ago

Thanks for taking the time to review. The additions in .gitignore via this commit allows developers to clearly see "what's really changed" in their local repos. The built products of make shouldn't interfere with that clear view. If, for example, I'm working on a source file and periodically building and testing, I'd like to see that I've changed only the source file, because in the end only those changes would be committed. It's cumbersome to make clean just to get that view. I'll mention that it's a typical custom in git use to ignore these product files.

terricain commented 8 years ago

That's cool, just wanted a justification ;). Cheers for the additions. If you can check from time to time, keeping it updated would be useful.