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: Update downstream SPL to program-specific repos #4371

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

joncinque
Copy link

Problem

SPL has been split up into multiple program-specific repos and will soon be archived, but the downstream jobs are still run against the SPL repo.

Summary of changes

Rather than use the SPL repo, check out the individual program-specific repos and run check, build-sbf, and test.

Once other PRs land, we can add back other programs and repos.

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.

LGTM! 👑

#### Problem

SPL has been split up into multiple program-specific repos and will soon
be archived, but the downstream jobs are still run against the SPL repo.

#### Summary of changes

Rather than use the SPL repo, check out the individual program-specific
repos and run `check`, `build-sbf`, and `test`.

Once other PRs land, we can add back other programs and repos.
@joncinque joncinque added the automerge automerge Merge this Pull Request automatically once CI passes label Jan 10, 2025
@joncinque joncinque merged commit 1383d53 into anza-xyz:master Jan 10, 2025
42 checks passed
@joncinque joncinque deleted the downstream-spl branch January 10, 2025 18:33
@t-nelson
Copy link

does this need backported?

@joncinque
Copy link
Author

v2.1 has the downstream jobs disabled already, so no https://github.com/anza-xyz/agave/blob/v2.1/.github/workflows/downstream-project-spl.yml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants