-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[SR-7074] Introduce ability to add top level files #1516
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
Conversation
|
||
// Add root level files to the project. | ||
func shouldIgnore(_ path: AbsolutePath) -> Bool { | ||
return path.basename.hasPrefix(".") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is more useful to ignore dotfiles. (like .gitignore
or .DS_Store
).
How do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like some files to be included like .swiftlint.yml
but that might be possible with a different flag such as --include .swiftlint.yml
I'll add tests after merging #1517. |
@ViktorSimko OK. Could you review #1517 ? |
I'm going to add some tests for this feature. |
It looks like this bug is dupe of SR-1741 and @ViktorSimko has already done a significant work in PR #1507 to resolve that issue. @giginet, do you mind picking up another starter bug 😊? |
Implement SR-7074.
I introduce the ability to add top level files (like
README.md
) to a generated Xcode project.$ swift package generate-xcodeproj