chore(release): prepare release v0.32.2#6643
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (3)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughVersion 0.32.2 release introducing a new public RPC method Forest.TraceCall for API v1 and v2, with changelog documentation and manifest version update. Existing parallel HTTP downloads feature for snapshots remains included. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
CHANGELOG.md (1)
58-58:⚠️ Potential issue | 🟡 MinorWording mismatch: "Added…" entry placed under
### Fixed.The description opens with "Added random GC delay…" which reads like an addition, not a fix. Either move it to
### Addedor rephrase to emphasize what was broken, e.g.:- - [`#6594`](https://github.com/ChainSafe/forest/issues/6594): Added random GC delay to avoid a cluster of nodes run GC and reboot RPC services at the same time. + + [`#6594`](https://github.com/ChainSafe/forest/issues/6594): Fixed GC clustering issue by introducing a random delay to prevent nodes from running GC and rebooting RPC services simultaneously.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@CHANGELOG.md` at line 58, The changelog entry "[`#6594`]: Added random GC delay to avoid a cluster of nodes run GC and reboot RPC services at the same time." is under "### Fixed" but reads like an addition; either move the entry to "### Added" or reword the entry under "### Fixed" to describe the problem being fixed (for example: "Introduced random GC delay to prevent clusters of nodes from running GC simultaneously and rebooting RPC services" or "Fixed simultaneous GC reboots by adding a random GC delay"). Update the line containing "[`#6594`]" accordingly so the section and wording match.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CHANGELOG.md`:
- Around line 38-40: Update the PR title and description to match the actual
release version and fix the typo: change the PR description text that currently
reads "prepare release v0.31.2 'Phagun'" to "prepare release v0.32.2 'Phagun'",
and correct the PR title's "rlease" typo; verify that the release string in
CHANGELOG.md ("Forest v0.32.2 \"Phagun\"") and Cargo.toml remain unchanged and
consistent with the updated PR text.
---
Outside diff comments:
In `@CHANGELOG.md`:
- Line 58: The changelog entry "[`#6594`]: Added random GC delay to avoid a
cluster of nodes run GC and reboot RPC services at the same time." is under "###
Fixed" but reads like an addition; either move the entry to "### Added" or
reword the entry under "### Fixed" to describe the problem being fixed (for
example: "Introduced random GC delay to prevent clusters of nodes from running
GC simultaneously and rebooting RPC services" or "Fixed simultaneous GC reboots
by adding a random GC delay"). Update the line containing "[`#6594`]" accordingly
so the section and wording match.
ℹ️ Review info
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
CHANGELOG.mdCargo.toml
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
Outside contributions
Summary by CodeRabbit
New Features
Chores