Skip to content
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

added codelite project files to the gitignore #7

Merged
merged 2 commits into from
May 20, 2013

Conversation

underdoeg
Copy link
Contributor

It's your call whether you want to merge this. I know that OF doesn't officially support codelite. But I don't think that those lines will conflict with any other IDE...

added codelite 
@bilderbuchi
Copy link
Member

  • Are all those user-specific IDE files? Cause currently, we package IDE files with addons because the PG can't create them yet (see Keep project files out of addons folders. #1) - we only ignore IDE project files which are specific to a user and should thus not be committed (e.g. C::B .layout files)
  • could you move your changes into the IDE-specific ignore patterns (e.g. user-specific files) section of the gitignore file?
  • From looking at it, the last rule correctly does not match codeblocks *.workspace files. Is that assumption correct?

@underdoeg
Copy link
Contributor Author

  • yes, those are user specific. sorry for the bad PR title...
  • Sure thing, I'll move them.
  • no it does not match .workspace codeblocks files. I know because codelite also uses a .workspace filename.

moved codelite block to ide specific section
@bilderbuchi
Copy link
Member

Alright then. We don't have a merge procedure here, but +1 to merge from me, and I'll let @benben merge if he doesn't have objections.
Thank you!

benben added a commit that referenced this pull request May 20, 2013
added codelite project files to the gitignore
@benben benben merged commit 86cae07 into openframeworks:master May 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants