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

(v1.17) CI: Checkout v1.17 branch from SPL for downstream job #108

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

joncinque
Copy link

Problem

As pointed out in #106, the SPL downstream job is failing CI on 1.17 because SPL is using 1.18.

Summary of Changes

When performing the downstream job, checkout branch v1.17 on the SPL repo, which is the last commit on 1.17.

Fixes #

@joncinque joncinque requested a review from yihau March 6, 2024 12:12
@joncinque joncinque changed the title CI: Checkout v1.17 branch from SPL for downstream job (v1.17) CI: Checkout v1.17 branch from SPL for downstream job Mar 6, 2024
Copy link
Member

@yihau yihau 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 to me! the pipeline didn't trigger correctly here. just checked manually and it worked fine!

@joncinque joncinque merged commit bc3ab0f into anza-xyz:v1.17 Mar 6, 2024
15 checks passed
@joncinque joncinque deleted the splci branch March 6, 2024 13:58
@CriesofCarrots
Copy link

Thanks, I didn't realize or had forgotten we did this for v1.16. Much nicer than ignoring altogether!

@HaoranYi HaoranYi mentioned this pull request Apr 8, 2024
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.

3 participants