Skip to content

sync up xcode related gitignore files #1514

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

Merged
merged 1 commit into from
May 30, 2015
Merged

Conversation

bbodenmiller
Copy link
Contributor

Xcode, Obj-C, & Swift updates. Also comment to help prevent fragmentation.

Some details based on #1221

@shiftkey
Copy link
Member

Is it worth having a discussion about unifying these ignore files? We did this a while ago for VisualStudio - which was a combination of CSharp and VisualBasic languages - and I can't help but feel this is heading down that path...

@arcresu
Copy link
Contributor

arcresu commented May 14, 2015

@shiftkey See comments on 5bd4330 for why we have the current state of these particular templates. I think it needs fixing, but people have strong feelings here.

@bbodenmiller
Copy link
Contributor Author

Can we get these changes merged? Given the comments on the other thread I don't see a merge of the templates being feasible in the near future.

@shiftkey
Copy link
Member

@bbodenmiller I'll give @arcresu a chance to chime in if has any issues with it, but I'm 👍 with this change.

@arcresu
Copy link
Contributor

arcresu commented May 30, 2015

Thanks, yeah this looks like a good change.

arcresu added a commit that referenced this pull request May 30, 2015
sync up xcode related gitignore files
@arcresu arcresu merged commit 28adf70 into github:master May 30, 2015
@bbodenmiller bbodenmiller deleted the xcode branch May 30, 2015 05:02
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
sync up xcode related gitignore files
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