Skip to content

ci: refactor CI tests to group fast tests in a single run#1780

Merged
netrome merged 2 commits intomainfrom
1779-consolidate-ci-actions
Jan 16, 2026
Merged

ci: refactor CI tests to group fast tests in a single run#1780
netrome merged 2 commits intomainfrom
1779-consolidate-ci-actions

Conversation

@gilcu3
Copy link
Contributor

@gilcu3 gilcu3 commented Jan 16, 2026

Closes #1779

@gilcu3 gilcu3 linked an issue Jan 16, 2026 that may be closed by this pull request
@gilcu3 gilcu3 force-pushed the 1779-consolidate-ci-actions branch 7 times, most recently from dfd2eda to 63acc7f Compare January 16, 2026 10:22
@gilcu3 gilcu3 force-pushed the 1779-consolidate-ci-actions branch from 63acc7f to df94896 Compare January 16, 2026 10:26
netrome
netrome previously approved these changes Jan 16, 2026
Copy link
Collaborator

@netrome netrome 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, but I think it would be even more clear if we had the default "fast" checks in ci.yml and instead only broke out the remaining slow checks (nextest, building docker image etc.) instead. Wdyt?

@gilcu3 gilcu3 force-pushed the 1779-consolidate-ci-actions branch from 88239fd to ccb77cf Compare January 16, 2026 11:05
Copy link
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the reorg!

with:
tool: cargo-make@0.37.24

- name: Run cargo-make fast checks
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit torn on the name "fast checks" but I don't have any good better suggestions. Perhaps "hygiene checks"? Or "sanity checks"? Eh "fast checks" still works.

@gilcu3 gilcu3 enabled auto-merge January 16, 2026 11:22
@DSharifi DSharifi changed the title test: refactor CI tests to group fast tests in a single run CI: refactor CI tests to group fast tests in a single run Jan 16, 2026
@DSharifi DSharifi changed the title CI: refactor CI tests to group fast tests in a single run ci: refactor CI tests to group fast tests in a single run Jan 16, 2026
@netrome netrome disabled auto-merge January 16, 2026 13:02
@netrome netrome merged commit 58593ba into main Jan 16, 2026
9 checks passed
@netrome netrome deleted the 1779-consolidate-ci-actions branch January 16, 2026 13:02
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.

consolidate CI actions

3 participants