chore: prepare release v0.33.2#6984
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
WalkthroughAdds Forest v0.33.2 "Night Mare" release notes (moves NV28/FireHorse calibnet item into the new release section with activation details), bumps crate version, updates devnet Lotus image, and replaces devnet actor bundle/manifest metadata to v18.0.0. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
✨ Simplify code
Review rate limit: 4/5 reviews remaining, refill in 12 minutes. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
scripts/devnet/.env (1)
1-1: PinLOTUS_IMAGEby digest to ensure reproducibility.RC tags are mutable and can drift between builds. Use the digest for the
linux/amd64platform instead:Suggested change
-LOTUS_IMAGE=filecoin/lotus-all-in-one:v1.36.0-rc1-2k +LOTUS_IMAGE=filecoin/lotus-all-in-one@sha256:a480ca9fd0dfee72c02db1aa3fb4bc5ec500344f17d63691e822c05be0801933🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@scripts/devnet/.env` at line 1, Replace the mutable RC tag value for LOTUS_IMAGE with a platform-specific digest to ensure reproducible builds: update the LOTUS_IMAGE environment variable (the LOTUS_IMAGE entry) to reference the image by its sha256 digest for linux/amd64 instead of the tag "filecoin/lotus-all-in-one:v1.36.0-rc1-2k", e.g., obtain the correct digest for linux/amd64 from the registry and set LOTUS_IMAGE=filecoin/lotus-all-in-one@sha256:<digest>.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@scripts/devnet/.env`:
- Line 1: Replace the mutable RC tag value for LOTUS_IMAGE with a
platform-specific digest to ensure reproducible builds: update the LOTUS_IMAGE
environment variable (the LOTUS_IMAGE entry) to reference the image by its
sha256 digest for linux/amd64 instead of the tag
"filecoin/lotus-all-in-one:v1.36.0-rc1-2k", e.g., obtain the correct digest for
linux/amd64 from the registry and set
LOTUS_IMAGE=filecoin/lotus-all-in-one@sha256:<digest>.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 53af9f40-9e40-4cc5-b8dc-275d83245286
⛔ Files ignored due to path filters (4)
Cargo.lockis excluded by!**/*.locksrc/rpc/snapshots/forest__rpc__tests__rpc__v0.snapis excluded by!**/*.snapsrc/rpc/snapshots/forest__rpc__tests__rpc__v1.snapis excluded by!**/*.snapsrc/rpc/snapshots/forest__rpc__tests__rpc__v2.snapis excluded by!**/*.snap
📒 Files selected for processing (3)
CHANGELOG.mdCargo.tomlscripts/devnet/.env
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
State migration matches on calibnet on an arbitrary epoch
Change checklist
Outside contributions
Summary by CodeRabbit
Bug Fixes
Chores
Documentation