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 ignore for the CocoaPods 'Pods' folder #418

Closed
wants to merge 1 commit into from
Closed

Added ignore for the CocoaPods 'Pods' folder #418

wants to merge 1 commit into from

Conversation

matzew
Copy link
Contributor

@matzew matzew commented Aug 21, 2012

No description provided.

@flyerhzm
Copy link

+1

@jspahrsummers
Copy link
Contributor

Thanks for submitting this! It looks like your forked repository was deleted, though. Can you please re-create this patch if you'd still like it merged?

@bartj3
Copy link

bartj3 commented Jul 2, 2013

The fork seems to be ok now, would be nice if this was included. I'd say CocoaPods is standard enough to deserve a spot in the gitignore file.

@matzew
Copy link
Contributor Author

matzew commented Jul 2, 2013

@jspahrsummers just created a new fork - sorry for the deletion:
https://github.com/matzew/gitignore

One question... we also have something like below, for the generated workspace:

*.xcworkspace
!default.xcworkspace

should I add that too ?

@jspahrsummers
Copy link
Contributor

@matzew That's already in there.

Thanks! 🌟

@matzew
Copy link
Contributor Author

matzew commented Jul 3, 2013

@jspahrsummers any comment on xcworkspace ?

@bartj3
Copy link

bartj3 commented Jul 3, 2013

The Podfile.lock file should not be in the ignore right? (like the Gemfile.lock) it ensures the correct podversions are used between different systems

@jspahrsummers
Copy link
Contributor

@matzew Check out the file, higher up. It's already ignored in there.

@bartj3 Ah, sorry, I didn't realize that. Fixed in 444416f.

@bartj3
Copy link

bartj3 commented Jul 3, 2013

thanks 👍

drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014
rvodden pushed a commit to rvodden/gitignore that referenced this pull request Apr 30, 2022
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.

4 participants