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: cleanup actions config #10

Merged
merged 1 commit into from
Feb 2, 2024
Merged

ci: cleanup actions config #10

merged 1 commit into from
Feb 2, 2024

Conversation

Noah-Kennedy
Copy link
Collaborator

This change cleans up our actions config to be less brittle when we make changes to CI, and to also be faster.

It uses several prebuilt actions from trusted providers for installing and using different tools, as well as caching artifacts. These changes should speed up our CI runs quite a bit.

It also reduces brittleness by specifying a new job, pr-status-checks, which we can specify PRs to depend on so that adding new jobs or editing matrixes won't break CI and require admin intervention.

It also breaks out our lint job into a set of smaller jobs to improve CI parallelism.

@Noah-Kennedy
Copy link
Collaborator Author

@inikulin this one will require manual intervention from an admin. After this, pr-status-checks will be the only CI job we need to depend on.

This change cleans up our actions config to be less brittle when we make changes to CI, and to also be faster.

It uses several prebuilt actions from trusted providers for installing and using different tools, as well as caching artifacts. These changes should speed up our CI runs quite a bit.

It also reduces brittleness by specifying a new job, `pr-status-checks`, which we can specify PRs to depend on so that adding new jobs or editing matrixes won't break CI and require admin intervention.

It also breaks out our lint job into a set of smaller jobs to improve CI parallelism.
@Noah-Kennedy Noah-Kennedy merged commit e2e21ec into main Feb 2, 2024
14 checks passed
@Noah-Kennedy Noah-Kennedy deleted the noah/cleanup-ci branch February 2, 2024 16:20
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