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

[CI] Use repo_name as cross-PR lookup rather than id #74884

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Jul 2, 2024

Rather than using the repo id (org/repository) to find the cross-PR testing branch, instead use the repo name. This is less likely to fail during the migration of apple -> swiftlang as it does not rely on updating the update-checkout config with the new repository id.

Rather than using the repo id (org/repository) to find the cross-PR
testing branch, instead use the repo name. This is less likely to fail
during the migration of `apple` -> `swiftlang` as it does not rely on
updating the `update-checkout` config with the new repository id.
@bnbarham bnbarham requested a review from shahmishal as a code owner July 2, 2024 04:59
@bnbarham
Copy link
Contributor Author

bnbarham commented Jul 2, 2024

@bnbarham
Copy link
Contributor Author

bnbarham commented Jul 2, 2024

[swift-driver]                          + git checkout ci_pr_988
[swift-driver]                          Switched to branch 'ci_pr_988'
...
[swift-syntax]                          + git checkout ci_pr_2201
[swift-syntax]                          Switched to branch 'ci_pr_2201'

Want to merge @shahmishal?

@bnbarham
Copy link
Contributor Author

bnbarham commented Jul 2, 2024

@swift-ci please smoke test

@shahmishal shahmishal merged commit 80e0e02 into swiftlang:main Jul 3, 2024
0 of 3 checks passed
@bnbarham bnbarham deleted the change-cross-pr-handling branch July 3, 2024 01:48
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