Skip to content

Move code for finding test targets into PackageGraph #2109

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 3, 2019

Conversation

neonichu
Copy link
Contributor

@neonichu neonichu commented May 3, 2019

Other clients might also want to know which test targets are relevant for a particular executable target.

rdar://problem/47648572

Other clients might also want to know which test targets are relevant
for a particular executable target.

rdar://problem/47648572
@neonichu neonichu requested a review from aciidgh May 3, 2019 00:14
@neonichu
Copy link
Contributor Author

neonichu commented May 3, 2019

@swift-ci please smoke test

1 similar comment
@neonichu
Copy link
Contributor Author

neonichu commented May 3, 2019

@swift-ci please smoke test

@aciidgh
Copy link
Contributor

aciidgh commented May 3, 2019

Can you add a unit test?

@neonichu neonichu merged commit 85cfe06 into swiftlang:master May 3, 2019
@neonichu neonichu deleted the move-test-target-computation branch May 3, 2019 22:36
neonichu added a commit to neonichu/swift-package-manager that referenced this pull request May 3, 2019
Other clients might also want to know which test targets are relevant for a particular executable target.

rdar://problem/47648572
aciidgh pushed a commit that referenced this pull request May 3, 2019
Other clients might also want to know which test targets are relevant for a particular executable target.

rdar://problem/47648572
aemino pushed a commit to aemino/swift-package-manager that referenced this pull request Sep 14, 2019
Other clients might also want to know which test targets are relevant for a particular executable target.

rdar://problem/47648572
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