Uh oh!
There was an error while loading. Please reload this page.
fix: wrong shell for mise task - #6433
Conversation
WalkthroughA single configuration property was added to the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@mise.toml`:
- Line 10: The shell property uses bash correctly but is formatted without
spaces; update the TOML property name/value formatting by changing shell="bash
-c" to shell = "bash -c" so it matches the file's style (e.g., similar to
description and run properties) and satisfies taplo fmt/lint checks.
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
mise.toml
🧰 Additional context used
📓 Path-based instructions (1)
**/*.toml
📄 CodeRabbit inference engine (AGENTS.md)
Ensure TOML files are formatted correctly using
taplo fmtand passtaplo lint
Files:
mise.toml
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
Repo: ChainSafe/forest PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-01-14T10:25:49.996Z
Learning: Ensure all tests pass with `mise test` before submitting pull requests
Learnt from: LesnyRumcajs
Repo: ChainSafe/forest PR: 5907
File: src/rpc/methods/state.rs:523-570
Timestamp: 2025-08-06T15:44:33.467Z
Learning: LesnyRumcajs prefers to rely on BufWriter's Drop implementation for automatic flushing rather than explicit flush() calls in Forest codebase.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
- GitHub Check: tests-release
- GitHub Check: rubocop
- GitHub Check: Build forest binaries on Linux AMD64
- GitHub Check: Build MacOS
- GitHub Check: Build Ubuntu
- GitHub Check: cargo-publish-dry-run
- GitHub Check: Coverage
- GitHub Check: All lint checks
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
Uh oh!
There was an error while loading. Please reload this page.
Summary of changes
Changes introduced in this pull request:
=> => # Installing Forest binaries with profile: release => => # sh: 2: [[: not foundwhen installing Forest.Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.