-
Notifications
You must be signed in to change notification settings - Fork 29
Release 25.07.1 #8775
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release 25.07.1 #8775
Conversation
📝 WalkthroughWalkthroughThis update documents the release of version 25.07.1, adding new entries to the changelog and migration guide. It consolidates previously unreleased change descriptions into the official changelog and removes corresponding temporary files. The migration guide notes no required database evolutions for this release. Changes
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:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
CHANGELOG.released.md (3)
13-15
: Entry in “Highlights” is overly verboseThis bullet is ~45 words long and mixes several caveats. Released-file entries are usually kept crisp; suggest trimming to the core value prop and moving the reload caveat to docs or a follow-up sentence.
31-31
: Tweak wording for clarity“Would crash directly” is awkward; replacing with “would crash immediately” reads better.
- Fixed that WEBKNOSSOS would crash directly if the communication with the server was interrupted while using the proofreading tool. + Fixed an issue where WEBKNOSSOS would crash immediately if communication with the server was interrupted during proofreading.
33-33
: Capitalisation: “3D”Use the common “3D” instead of “3d” for consistency with previous entries.
- Fixed broken node selection in 3d viewport when continuous drawing was enabled in skeleton tool. + Fixed broken node selection in 3D viewport when continuous drawing was enabled in the skeleton tool.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (16)
CHANGELOG.released.md
(1 hunks)MIGRATIONS.released.md
(2 hunks)unreleased_changes/8648.md
(0 hunks)unreleased_changes/8711.md
(0 hunks)unreleased_changes/8717.md
(0 hunks)unreleased_changes/8735.md
(0 hunks)unreleased_changes/8738.md
(0 hunks)unreleased_changes/8743.md
(0 hunks)unreleased_changes/8756.md
(0 hunks)unreleased_changes/8761.md
(0 hunks)unreleased_changes/8767.md
(0 hunks)unreleased_changes/8768.md
(0 hunks)unreleased_changes/8769.md
(0 hunks)unreleased_changes/8771.md
(0 hunks)unreleased_changes/8772.md
(0 hunks)unreleased_changes/8773.md
(0 hunks)
💤 Files with no reviewable changes (14)
- unreleased_changes/8768.md
- unreleased_changes/8761.md
- unreleased_changes/8717.md
- unreleased_changes/8773.md
- unreleased_changes/8769.md
- unreleased_changes/8743.md
- unreleased_changes/8738.md
- unreleased_changes/8735.md
- unreleased_changes/8771.md
- unreleased_changes/8767.md
- unreleased_changes/8772.md
- unreleased_changes/8648.md
- unreleased_changes/8756.md
- unreleased_changes/8711.md
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: MichaelBuessemeyer
PR: scalableminds/webknossos#8221
File: CHANGELOG.unreleased.md:14-14
Timestamp: 2024-11-25T14:52:23.201Z
Learning: For the `webknossos` project, changelog entries in `CHANGELOG.unreleased.md` should be concise and avoid unnecessary detailed explanations.
Learnt from: philippotto
PR: scalableminds/webknossos#8654
File: CHANGELOG.released.md:10-12
Timestamp: 2025-06-02T09:34:16.064Z
Learning: In the `CHANGELOG.released.md` file, the release date may intentionally reflect the intended publication month (matching the version), even if the release was created earlier. This is an accepted workflow for this project.
MIGRATIONS.released.md (1)
Learnt from: MichaelBuessemeyer
PR: scalableminds/webknossos#8221
File: CHANGELOG.unreleased.md:14-14
Timestamp: 2024-11-25T14:52:23.201Z
Learning: For the `webknossos` project, changelog entries in `CHANGELOG.unreleased.md` should be concise and avoid unnecessary detailed explanations.
CHANGELOG.released.md (2)
undefined
<retrieved_learning>
Learnt from: MichaelBuessemeyer
PR: #8221
File: CHANGELOG.unreleased.md:14-14
Timestamp: 2024-11-25T14:52:23.201Z
Learning: For the webknossos
project, changelog entries in CHANGELOG.unreleased.md
should be concise and avoid unnecessary detailed explanations.
</retrieved_learning>
<retrieved_learning>
Learnt from: philippotto
PR: #8654
File: CHANGELOG.released.md:10-12
Timestamp: 2025-06-02T09:34:16.064Z
Learning: In the CHANGELOG.released.md
file, the release date may intentionally reflect the intended publication month (matching the version), even if the release was created earlier. This is an accepted workflow for this project.
</retrieved_learning>
⏰ 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). (3)
- GitHub Check: backend-tests
- GitHub Check: build-smoketest-push
- GitHub Check: frontend-tests
🔇 Additional comments (2)
MIGRATIONS.released.md (1)
9-14
: Release entry looks correct and consistent– Version/tag, compare-link, and date follow the established pattern.
– “None.” for Postgres evolutions aligns with previous releases that required no DB changes.
No action required.CHANGELOG.released.md (1)
16-18
: Double use of PR #8711 in both “Added” and “Fixed” – verify intentIf the same PR truly delivered both a new feature (zarr3/object-store support) and a bug-fix for mag1 handling, keep both bullets but consider an explicit note (“same PR”) or split the PR internally to avoid reader confusion.
Also applies to: 26-27
Cut at 1562965.
(Please delete unneeded items, merge only when none are left open)
$PR_NUMBER.md
file inunreleased_changes
or use./tools/create-changelog-entry.py
)