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

Excludes XC(UI)Test files from regular targets #397

Merged
merged 2 commits into from
Apr 20, 2019
Merged

Excludes XC(UI)Test files from regular targets #397

merged 2 commits into from
Apr 20, 2019

Conversation

ChristianSteffens
Copy link
Contributor

This PR removes some XC(UI)Test related files from the regular targets to fix some build errors.
I'm confused, did nobody else noticed this already or am I'm missing something?

@Vkt0r
Copy link
Member

Vkt0r commented Apr 20, 2019

Hey, @ChristianSteffens Thanks for taking the time to create this! Yes, I noticed recently and I fixed in my local 😅.

@Vkt0r
Copy link
Member

Vkt0r commented Apr 20, 2019

Can you please add an entry to the CHANGELOG.md following the guides there and I would be happy to merge the PR.

@ChristianSteffens
Copy link
Contributor Author

Sure thing.

Copy link
Member

@Vkt0r Vkt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@Vkt0r Vkt0r merged commit 7e07b39 into httpswift:stable Apr 20, 2019
tomieq pushed a commit to tomieq/swifterfork that referenced this pull request Apr 1, 2021
Excludes XC(UI)Test files from regular targets
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.

2 participants