Unpin nightly channel for miri tests by ShaharNaveh · Pull Request #7378 · RustPython/RustPython
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
📒 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
- Unpin Rust nightly in CI #6022: Modifies the same miri job to use the generic "nightly" Rust toolchain via the NIGHTLY_CHANNEL variable.
- fix miri ci #7093: Changes the same NIGHTLY_CHANNEL environment variable in the miri job (updates its value).
- Pin rustc nightly to
nightly-2025-07-18for miri tests #5999: Modifies how the miri job's Rust nightly toolchain is specified in the CI workflow configuration.
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.
Comment @coderabbitai help to get the list of available commands and usage tips.