Skip to content

ci: pin GitHub Actions to commit SHA#536

Open
sobhe-Starkware wants to merge 2 commits into
mainfrom
pin-github-actions-to-sha
Open

ci: pin GitHub Actions to commit SHA#536
sobhe-Starkware wants to merge 2 commits into
mainfrom
pin-github-actions-to-sha

Conversation

@sobhe-Starkware

Copy link
Copy Markdown

No description provided.

@sobhe-Starkware sobhe-Starkware changed the title pin GitHub Actions to commit SHA ci: pin GitHub Actions to commit SHA Jul 26, 2026
sobhe-Starkware and others added 2 commits July 26, 2026 13:53
The clippy, run-tests and coverage jobs used dtolnay/rust-toolchain's
floating `stable`. Current stable fails to build cairo-vm 1.0.0-rc4,
which calls `num_integer::Integer::is_multiple_of(&self, &other)` — the
inherent `uint::is_multiple_of` stabilized in Rust 1.87 now shadows it
and rejects the borrowed argument, so the workspace no longer compiles.

Pin the same toolchain the workspace was written against; verified
locally that `scripts/clippy.sh` and `cargo test` pass on 1.79.0.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@sobhe-Starkware
sobhe-Starkware force-pushed the pin-github-actions-to-sha branch from 780653f to 39c580f Compare July 26, 2026 10:54
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.21%. Comparing base (93de0bd) to head (39c580f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #536   +/-   ##
=======================================
  Coverage   81.21%   81.21%           
=======================================
  Files          42       42           
  Lines        1826     1826           
  Branches     1826     1826           
=======================================
  Hits         1483     1483           
  Misses        269      269           
  Partials       74       74           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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