Skip to content

Add a test case for static products with a binary artifact dependency #8212

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
Jan 17, 2025

Conversation

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Jan 14, 2025

Add a test for #8055 that checks we can successfully generate a manifest for a target -> static product -> binary artifact dependency.

@MaxDesiatov
Copy link
Contributor

@swift-ci test

Copy link
Contributor

@yyvch yyvch left a comment

Choose a reason for hiding this comment

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

LGTM

@bnbarham
Copy link
Contributor Author

FWIW I didn't test as it is going to fail without the other change 😅. I'll test once it's in (though it seems to be blocked on spurious failures...)

@bnbarham
Copy link
Contributor Author

@swift-ci please test

Add a test
swiftlang#8055 that
checks we can successfully generate a manifest for a target -> static product
-> binary artifact dependency.
@bnbarham bnbarham force-pushed the product-with-artifact-dep branch from 86d7dba to a11477e Compare January 16, 2025 00:37
@bnbarham
Copy link
Contributor Author

@swift-ci please test

@bnbarham
Copy link
Contributor Author

@swift-ci please test Windows platform

@bnbarham bnbarham merged commit bd1cf1b into swiftlang:main Jan 17, 2025
5 checks passed
@bnbarham bnbarham deleted the product-with-artifact-dep branch January 17, 2025 02:31
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.

4 participants