Skip to content

[DO NOT MERGE] Test commit to cross check swift-driver change #7125

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

cachemeifyoucan
Copy link
Contributor

Cross check swiftlang/swift-driver#1488 to make sure it doesn't break swiftpm

@cachemeifyoucan
Copy link
Contributor Author

Please test with following pull request:
swiftlang/swift-driver#1488

@swift-ci Please test

@MaxDesiatov
Copy link
Contributor

I'm not entirely sure if this repo supports cross-repo testing, maybe worth creating one more PR at apple/swift and run tests there just to double check?

@cachemeifyoucan
Copy link
Contributor Author

I'm not entirely sure if this repo supports cross-repo testing, maybe worth creating one more PR at apple/swift and run tests there just to double check?

Creating working copy for https://github.com/apple/swift-driver.git
Working copy of https://github.com/apple/swift-driver.git resolved at main (916bf4b)

Indeed, seems no cross repo testing. Just double check what tests from apple/swift has this coverage? If I just start a full test with the swift-driver PR, it will cover swiftpm build?
Also I remember swiftpm builds in Test Source Compatibility test too, maybe I can start that as well.

@MaxDesiatov
Copy link
Contributor

@shahmishal would you have a moment to clarify what's the best way to test this as a cross-repo PR?

@cachemeifyoucan
Copy link
Contributor Author

Try swift repo here: swiftlang/swift#70051

@cachemeifyoucan
Copy link
Contributor Author

The other PR is testing cross repo correctly and I verify all the configuration I broke last time builds correctly now. Closing this.

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