Skip to content

[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

Closed
wants to merge 1 commit into from

Conversation

giginet
Copy link
Contributor

@giginet giginet commented Mar 3, 2018

Implement SR-7074.

I introduce the ability to add top level files (like README.md) to a generated Xcode project.

$ swift package generate-xcodeproj


// Add root level files to the project.
func shouldIgnore(_ path: AbsolutePath) -> Bool {
return path.basename.hasPrefix(".")
Copy link
Contributor Author

@giginet giginet Mar 3, 2018

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?

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

@giginet giginet changed the title Introduce ability to add top level files [SR-7074] Introduce ability to add top level files Mar 3, 2018
@giginet giginet changed the title [SR-7074] Introduce ability to add top level files [WIP] [SR-7074] Introduce ability to add top level files Mar 3, 2018
@giginet
Copy link
Contributor Author

giginet commented Mar 3, 2018

I'll add tests after merging #1517.

@ViktorSimko
Copy link
Contributor

This functionality is already implemented as a part of #1507 . Merging #1517 would also make the testing of that PR easier.

@giginet
Copy link
Contributor Author

giginet commented Mar 4, 2018

@ViktorSimko OK. Could you review #1517 ?

@hartbit
Copy link
Contributor

hartbit commented Jun 11, 2018

@giginet Now that #1517 has been merged, do you need anything else, or should this PR be reviewed?

@hartbit hartbit added the WIP Work in progress label Jun 12, 2018
@giginet giginet changed the title [WIP] [SR-7074] Introduce ability to add top level files [SR-7074] Introduce ability to add top level files Jun 14, 2018
@giginet
Copy link
Contributor Author

giginet commented Jun 14, 2018

I'm going to add some tests for this feature.
However, can you check this approach is good before?

@aciidgh
Copy link
Contributor

aciidgh commented Jun 28, 2018

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 😊?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants