Prepare Release of 25.04.0 #8549
Conversation
📝 WalkthroughWalkthroughThis change finalizes the release documentation for version 25.04.0, adding new entries to the changelog and migration guides to reflect features, fixes, performance improvements, and breaking changes introduced in this version. The unreleased changelog and migration documents are cleared of prior entries and updated to compare against the new release. Notable updates include mesh opacity control, dataset mirroring, backend and frontend dependency upgrades, a switch to Vitest testing framework requiring Node.js 22+, and removal of the top-level Changes
Possibly related PRs
Suggested reviewers
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
MIGRATIONS.released.md (2)
12-12: Use correct spelling for Node.js
The official name is “Node.js”, not “NodeJs”. Please update the migration note.- NodeJs version `22+` is required. [#8479](https://github.com/scalableminds/webknossos/pull/8479) + Node.js version `22+` is required. [#8479](https://github.com/scalableminds/webknossos/pull/8479)🧰 Tools
🪛 LanguageTool
[uncategorized] ~12-~12: The official spelling of this programming framework is “Node.js”.
Context: ...m//pull/8460) - NodeJs version22+is required. [#8479](http...(NODE_JS)
14-14: Remove trailing punctuation from heading
Markdown headings should not end with punctuation. Please drop the colon.- ### Postgres Evolutions: + ### Postgres Evolutions🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
14-14: Trailing punctuation in heading
Punctuation: ':'(MD026, no-trailing-punctuation)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
CHANGELOG.released.md(1 hunks)CHANGELOG.unreleased.md(1 hunks)MIGRATIONS.released.md(1 hunks)MIGRATIONS.unreleased.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
MIGRATIONS.released.md
[uncategorized] ~12-~12: The official spelling of this programming framework is “Node.js”.
Context: ...m//pull/8460) - NodeJs version 22+ is required. [#8479](http...
(NODE_JS)
🪛 markdownlint-cli2 (0.17.2)
MIGRATIONS.released.md
14-14: Trailing punctuation in heading
Punctuation: ':'
(MD026, no-trailing-punctuation)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: frontend-tests
- GitHub Check: build-smoketest-push
- GitHub Check: backend-tests
🔇 Additional comments (8)
CHANGELOG.released.md (6)
10-11: LGTM - Release entry has proper version and date formatThe version number and date are properly formatted, and the commits comparison link is correctly structured.
13-16: LGTM - Well-documented highlights sectionThe highlights section effectively summarizes the most important changes in this release, with proper PR references.
17-23: LGTM - Clear additions with appropriate referencesThe added features and improvements are well-documented with corresponding PR links.
24-31: LGTM - Changed behaviors properly documentedChanged behaviors and improvements are clearly described with appropriate PR references.
32-43: LGTM - Bug fixes well-documentedThe fixed issues are clearly described with appropriate PR references, providing users with clear information about what has been addressed in this release.
44-46: LGTM - Breaking change properly documentedThe breaking change regarding docker-compose.yml relocation is clearly identified with the corresponding PR reference.
MIGRATIONS.released.md (1)
9-9: Version mismatch between PR and migration guide
The migration guide documents release 25.04.1, but the PR title/summary indicates preparing release 25.04.0. Please confirm the intended release version.MIGRATIONS.unreleased.md (1)
9-9: Verify commit comparison range
The unreleased migration guide now compares from25.04.1...HEAD. Confirm this aligns with your intended next release (currently stated as 25.04.0 in PR summary) or update accordingly.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
CHANGELOG.released.md(1 hunks)CHANGELOG.unreleased.md(1 hunks)MIGRATIONS.released.md(1 hunks)MIGRATIONS.unreleased.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- MIGRATIONS.unreleased.md
🚧 Files skipped from review as they are similar to previous changes (2)
- CHANGELOG.unreleased.md
- CHANGELOG.released.md
🧰 Additional context used
🪛 LanguageTool
MIGRATIONS.released.md
[uncategorized] ~12-~12: The official spelling of this programming framework is “Node.js”.
Context: ...m//pull/8460) - NodeJs version 22+ is required. [#8479](http...
(NODE_JS)
🪛 markdownlint-cli2 (0.17.2)
MIGRATIONS.released.md
14-14: Trailing punctuation in heading
Punctuation: ':'
(MD026, no-trailing-punctuation)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: backend-tests
- GitHub Check: frontend-tests
- GitHub Check: build-smoketest-push
🔇 Additional comments (2)
MIGRATIONS.released.md (2)
9-10: Release entry formatting looks good
The new release heading for 25.04.0 and its commit comparison link are correctly formatted and positioned before the previous 25.03.1 entry.
11-11: FossilDB version requirement is accurate
The specified FossilDB version0.1.37and Docker Hub tagmaster__525:correctly reflect the breaking change introduced in PR #8460.
| ## [25.04.0](https://github.com/scalableminds/webknossos/releases/tag/25.04.0) - 2025-04-22 | ||
| [Commits](https://github.com/scalableminds/webknossos/compare/25.03.1...25.04.0) | ||
| - New FossilDB version `0.1.37` (`master__525:` on dockerhub) is required. [#8460](https://github.com/scalableminds/webknossos/pull/8460) | ||
| - NodeJs version `22+` is required. [#8479](https://github.com/scalableminds/webknossos/pull/8479) |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Correct Node.js naming and verify cleanup of unreleased guide
The term “NodeJs” should be updated to “Node.js” to match the official spelling. Additionally, please ensure that these released requirements (FossilDB 0.1.37 and Node.js 22+) have been removed from MIGRATIONS.unreleased.md.
- NodeJs version `22+` is required. [#8479]
+ Node.js version `22+` is required. [#8479]To verify that the unreleased guide no longer contains these entries, you can run:
🏁 Script executed:
#!/bin/bash
# Verify MIGRATIONS.unreleased.md no longer mentions the released requirements
rg -n "FossilDB version.*0\.1\.37" MIGRATIONS.unreleased.md
rg -n "NodeJs version.*22" MIGRATIONS.unreleased.mdLength of output: 115
🏁 Script executed:
#!/bin/bash
# List all migration guide markdown files
fd --type f 'MIGRATIONS.*\.md'
# If the unreleased guide exists, search for any lingering references
if [ -f MIGRATIONS.unreleased.md ]; then
echo "=== Searching in MIGRATIONS.unreleased.md ==="
rg -n '0\.1\.37' MIGRATIONS.unreleased.md
rg -n -i 'FossilDB' MIGRATIONS.unreleased.md
rg -n '22\+' MIGRATIONS.unreleased.md
rg -n -i 'Node' MIGRATIONS.unreleased.md
else
echo "MIGRATIONS.unreleased.md not found"
fiLength of output: 389
Spelling correction in released migration guide
Please update the Node.js spelling in MIGRATIONS.released.md (line 12):
- NodeJs version `22+` is required. [#8479]
+ Node.js version `22+` is required. [#8479]MIGRATIONS.unreleased.md has been verified and no longer contains any references to FossilDB 0.1.37 or NodeJs 22+, so no further cleanup is needed there.
Committable suggestion skipped: line range outside the PR's diff.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~12-~12: The official spelling of this programming framework is “Node.js”.
Context: ...m//pull/8460) - NodeJs version 22+ is required. [#8479](http...
(NODE_JS)
2 participants
(Please delete unneeded items, merge only when none are left open)