Repository navigation
Backport release/v6.7: Update changelog in prep to cut v6.7 - #4113
Conversation
|
Please cherry-pick the changes locally and resolve any conflicts. git fetch origin backport-4110-to-release/v6.7
git worktree add --checkout .worktree/backport-4110-to-release/v6.7 backport-4110-to-release/v6.7
cd .worktree/backport-4110-to-release/v6.7
git reset --hard HEAD^
git cherry-pick -x a665891ddca10fd7c1712972161ad0c1235f54d3
git push --force-with-lease |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Systematic changelog re-generation in prep to cut the v6.7 release. Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit a665891)
156572e to
ca48a69
Compare
PR SummaryLow Risk Overview This is a documentation-only backport of changelog generation ahead of tagging Reviewed by Cursor Bugbot for commit ca48a69. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Changelog-only backport that renames ## Unreleased to ## v6.7 and appends the full v6.7 PR list. Entries are well-formed (links match their PR numbers, strictly descending, no duplicates within the list); the only note is that the appended list has no heading, so it renders as part of the preceding ### Upgrade guide section.
Findings: 0 blocking | 2 non-blocking | 1 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- None at the file/PR level.
- 1 suggestion(s)/nit(s) flagged inline on specific lines.
- 1 non-blocking pre-existing issue(s) listed below under pre-existing issues.
Pre-existing issues
- [suggestion] CHANGELOG.md:46 (unchanged context in this PR) tells clients they "can enable
sei_getTransactionReceipt" for synthetic receipts, but that method is not inseiLegacyGatedMethods(evmrpc/sei_legacy.go:42, which only gatessei_getCosmosTx,sei_getEVMAddress,sei_getSeiAddress), so no[evm] enabled_legacy_sei_apisvalue can restore it. Following the migration advice cannot work.
Inline comments (could not post inline; listed here)
CHANGELOG.md:48(RIGHT) -- [suggestion] This baresei-chainline and the ~280 entries under it are appended directly after the### Upgrade guidesection, so in rendered Markdown the whole PR list reads as content of the upgrade guide. In the## v6.6section (line 329) the samesei-chainlist works because that release has no###subsections above it. Either give this list its own heading (e.g.### All changes) or move it above### Improvements/### Upgrade guideso the curated sections stay distinct from the generated list.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release/v6.7 #4113 +/- ##
================================================
- Coverage 61.34% 60.30% -1.04%
================================================
Files 2163 2064 -99
Lines 188767 177084 -11683
================================================
- Hits 115801 106798 -9003
+ Misses 62247 60509 -1738
+ Partials 10719 9777 -942
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
@seidroid review |
There was a problem hiding this comment.
Changelog-only backport that renames ## Unreleased to ## v6.7 and appends the generated v6.7 PR list; the entries are well-formed, strictly descending, duplicate-free, and correctly scoped to this release branch. The only note is the previously reported missing heading for the appended list, which the current revision still has.
Findings: 0 blocking | 2 non-blocking | 1 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- None at the file/PR level.
- 1 suggestion(s)/nit(s) flagged inline on specific lines.
- 1 non-blocking pre-existing issue(s) listed below under pre-existing issues.
Pre-existing issues
- [suggestion] CHANGELOG.md:46 (unchanged context in this PR) tells clients they "can enable
sei_getTransactionReceipt" for synthetic receipts, but that method is not inseiLegacyGatedMethods(evmrpc/sei_legacy.go:42, which gates onlysei_getCosmosTx,sei_getEVMAddress,sei_getSeiAddress), so no[evm] enabled_legacy_sei_apisvalue can restore it. Following the migration advice cannot work.
Inline comments (could not post inline; listed here)
CHANGELOG.md:48(RIGHT) -- [suggestion] Re-raising: this baresei-chainline and the 278 entries under it are appended directly after the### Upgrade guidesection, so in rendered Markdown the whole generated PR list reads as content of the upgrade guide. In the## v6.6section (line 328) the samesei-chainlist works because that release has no###subsections above it. Either give this list its own heading (e.g.### All changes) or move it above### Improvements/### Upgrade guideso the curated sections stay distinct from the generated list.
Adds the `release/v6.7` entries merged since the rc1 changelog (#4110), in prep to cut **v6.7.0-rc2**: - [#4292](#4292) — Flush MemIAVL changelog before exiting on an upgrade panic - [#4285](#4285) — fix(seidb): refuse a corrupted changelog in digest replay instead of repairing it - [#4255](#4255) — feat(seidb): Add JSON output to evm-logical-digest and inspect a FlatKV migration in flight - [#4116](#4116) — rc1 version bump - [#4113](#4113) — rc1 changelog backport Regenerated with `./scripts/generate-changelog.sh release/v6.6 release/v6.7`; only the `## v6.7` PR list changes, so the `backport release/v6.7` cherry-pick applies cleanly (verified with `git apply --check` against `origin/release/v6.7`). Docs-only; no code change. Made with [Cursor](https://cursor.com) Co-authored-by: Cursor <cursoragent@cursor.com>
Backport of #4110 to
release/v6.7.