Skip to content

CI: Re-enable downstream SPL job#3510

Merged
joncinque merged 2 commits intoanza-xyz:masterfrom
joncinque:downstream
Nov 7, 2024
Merged

CI: Re-enable downstream SPL job#3510
joncinque merged 2 commits intoanza-xyz:masterfrom
joncinque:downstream

Conversation

@joncinque
Copy link
Copy Markdown

Problem

The downstream SPL job was disabled in #3498 because the moved crates weren't correctly picked up in the SPL's patch script. The SPL patch script was fixed in
solana-labs/solana-program-library#7465, but the jobs are still disabled.

Summary of changes

Re-enable the SPL downstream jobs.

Side note: should all of the jobs have the if: github.repository == 'anza-xyz/agave' part?

#### Problem

The downstream SPL job was disabled in solana-labs#3498 because the moved crates
weren't correctly picked up in the SPL's patch script. The SPL patch
script was fixed in
solana-labs/solana-program-library#7465, but the
jobs are still disabled.

#### Summary of changes

Re-enable the SPL downstream jobs.
@joncinque joncinque requested a review from yihau November 6, 2024 23:22
@yihau
Copy link
Copy Markdown
Member

yihau commented Nov 7, 2024

yeah, could you add if: github.repository == 'anza-xyz/agave' to all jobs?

@joncinque
Copy link
Copy Markdown
Author

Ok, done!

Copy link
Copy Markdown
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.

🫶

@joncinque joncinque merged commit 00432e3 into anza-xyz:master Nov 7, 2024
@joncinque joncinque deleted the downstream branch November 7, 2024 14:39
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