Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.
Swift: ignore .swiftpm #3083
Swift: ignore .swiftpm #3083
Conversation
| @@ -38,7 +38,9 @@ playground.xcworkspace | |||
| # Packages/ | |||
| # Package.pins | |||
| # Package.resolved | |||
| # *.xcodeproj | |||
Coeur
Jul 23, 2019
Contributor
Really no need to give the wrong idea here: xcodeproj files are very needed in almost all the iOS/macOS projects of the world. So I wouldn't add this commented line.
maxxfrazer
Jul 23, 2019
Author
Contributor
This is specifically for swift packages; you open the Package.swift file with Xcode and all your code is brought in through that. The xcodeproj file can generated using swift package generate-xcodeproj.
If you're still not convinced though I'll remove the line.
Coeur
Jul 23, 2019
Contributor
Currently, I'm not used enough to Xcode 11 beta SPM to answer if there are better alternatives. I've ping previous committers.
|
@FabrizioBrancati @natestedman Can you review to tell if |
|
I found some interesting links:
It seems that adding this file to Maybe if it's commented by default, we can keep it. |
|
Ok, seems like something a lot of developers might want to ignore, especially if they're creating a package without dependencies - as Ankit has added it to the SPM Package. I'll move this up a line to group it with the other commented suggestions & comment it out. Do you think leaving a comment message would be necessary or just leave it there? Based on the InitPackage.swift file I think it would be appropriate to leave |
|
Yes, leave a comment to explain when and why the developer has to uncomment that line. |
|
@FabrizioBrancati should be all set if the added comment is to satisfaction |
|
@maxxfrazer thanks for the contribution, and thanks @Coeur @FabrizioBrancati for the reviews! |
|
@Coeur no problem - just catching up on notifications during a quiet morning, so I'll prioritize that next |
* upstream/master: Added Simulink cache files (github#3148) Add esp-idf.gitignore (github#3150) Add *.sch-bak to KiCad.gitignore (github#3153) Reference why to exclude binary jar, fixes github#111 (github#3154) Update Autotools.gitignore (github#3005) Improving the documentation of Objective-C and Swift files (github#3133) Ignore gummi previews (github#3144) Swift: ignore .swiftpm (github#3083) Update Godot.gitignore (github#3142) Revert of github#2988. TextMesh Pro files are not auto managed (github#3134) Partial revert of 837181e (github#3132) Added more glossaries auxiliary files and makeindex logs to TeX template (github#3104) .travis.yml: Trim trailing whitespace and indent (github#3118) Add *.hie to Haskell.gitignore (github#3120) Ignore the .ionide working folder (github#3108)
* added files to be ignored with swift package manager * added comment for swiftpm and why it might want to be included in .gitignore
Reasons for making this change:
SPM now supported with Xcode 11; .swiftpm is generated by Xcode and is not needed on git
Links to documentation supporting these rule changes:
Basing off Apple's swift-package-manager .gitignore
https://github.com/apple/swift-package-manager/blob/master/.gitignore