Skip to content

Unpin nightly channel for miri tests#7378

Merged
youknowone merged 1 commit intoRustPython:mainfrom
ShaharNaveh:ci-unpin-miri-nightly
Mar 7, 2026
Merged

Unpin nightly channel for miri tests#7378
youknowone merged 1 commit intoRustPython:mainfrom
ShaharNaveh:ci-unpin-miri-nightly

Conversation

@ShaharNaveh
Copy link
Contributor

@ShaharNaveh ShaharNaveh commented Mar 7, 2026

Summary by CodeRabbit

  • Chores
    • Updated continuous integration testing configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: fbbdcb9b-124a-4344-832a-54cedb2a69c7

📥 Commits

Reviewing files that changed from the base of the PR and between ed5bffe and bc8e3a6.

📒 Files selected for processing (1)
  • .github/workflows/ci.yaml

📝 Walkthrough

Walkthrough

Updated the NIGHTLY_CHANNEL environment variable in the miri job from nightly-2026-02-11 (a specific dated version) to nightly (a generic nightly channel), allowing the CI workflow to use the latest available Rust nightly toolchain.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yaml
Updated miri job's NIGHTLY_CHANNEL from a pinned dated version (nightly-2026-02-11) to the generic nightly channel.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • youknowone

Poem

🐰 Hop, hop—to nightly we go!
No dates to constrain our flow,
The miri job now runs so free,
With latest toolchains, wild and carefree!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Unpin nightly channel for miri tests' accurately describes the main change: updating the miri job's NIGHTLY_CHANNEL from a pinned version (nightly-2026-02-11) to the unpinned nightly channel.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@youknowone youknowone merged commit d9020c5 into RustPython:main Mar 7, 2026
11 of 13 checks passed
youknowone pushed a commit to youknowone/RustPython that referenced this pull request Mar 8, 2026
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