Skip to content

ci: trigger macos clippy after linux check passed#244

Closed
yihau wants to merge 3 commits intoanza-xyz:masterfrom
yihau:trigger-macos-clippy-by-buildkite
Closed

ci: trigger macos clippy after linux check passed#244
yihau wants to merge 3 commits intoanza-xyz:masterfrom
yihau:trigger-macos-clippy-by-buildkite

Conversation

@yihau
Copy link
Copy Markdown
Member

@yihau yihau commented Mar 14, 2024

Problem

We use large runners for macos clippy. would like to have another process for saving money. 💸

Summary of Changes

call macos clippy after linux check.

(--fail-with-body is added in 7.76.0 😞 so I use --fail instead)

@yihau yihau force-pushed the trigger-macos-clippy-by-buildkite branch 2 times, most recently from 32cd8ff to 1edde1a Compare March 14, 2024 15:26
@yihau yihau force-pushed the trigger-macos-clippy-by-buildkite branch from 1edde1a to ae13e45 Compare March 14, 2024 16:41
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.9%. Comparing base (794cb2f) to head (68d8ff4).
Report is 11 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         836      836           
  Lines      226643   226643           
=======================================
+ Hits       185636   185664   +28     
+ Misses      41007    40979   -28     

@yihau yihau marked this pull request as ready for review March 15, 2024 16:47
@yihau
Copy link
Copy Markdown
Member Author

yihau commented Mar 19, 2024

it looks like:
Screenshot 2024-03-19 at 14 27 43

when this one get merged, we will only trigger macos clippy when linux check are all green

(it will fail in this PR due to we haven't had this pipeline)

Copy link
Copy Markdown

@CriesofCarrots CriesofCarrots left a comment

Choose a reason for hiding this comment

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

🪖

OliverNChalk pushed a commit to OliverNChalk/agave that referenced this pull request Nov 11, 2025
@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the stale label Jan 12, 2026
@yihau yihau closed this Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants