Update release notes for 1.7.5 - #2699
Conversation
|
@claude review this PR |
|
Claude finished @sejas's task in 1m 38s —— View job PR Review: Update release notes for 1.7.5
OverviewThis PR replaces auto-generated release notes (raw commit titles) with curated, user-friendly entries for 1.7.5. The curation approach — removing internal tooling changes, dependency bumps, and test fixes — is consistent with how prior releases like 1.7.4 were written. Overall the content reads well. Issues FoundStyle inconsistency — trailing period (line 19) One bullet ends with a period while all others do not: All other entries in 1.7.5 (and throughout the entire file) omit trailing punctuation. The period should be removed. Potentially Missing User-Facing ItemsA few commits that landed in the release branch appear user-facing but are not in the notes. These may have been intentionally omitted — worth a quick check:
These are low-severity observations — if they were intentionally scoped out (e.g., shipped as internal improvements, or deferred), the current notes are fine. What's Working Well
SummaryOne minor fix needed: remove the trailing period on line 19. A second pass on whether the Blueprint login (#2527) and CLI interactive create (#2657) warrant a mention is also recommended, as both are directly user-facing changes. |
There was a problem hiding this comment.
Pull request overview
Updates the project’s release notes to reflect the shipped changes in version 1.7.5, keeping end-user documentation current alongside the codebase.
Changes:
- Replaced the verbose 1.7.5 changelog list with a concise, user-focused summary of key features, fixes, and improvements.
- Consolidated dependency update items into a single high-level entry.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * Fixed toggle sidebar icon alignment after Add Site modal #2636 | ||
| * Improved accessibility attributes in site settings #2618, #2654 | ||
| * Improved compatibility with Studio sites with legacy mu-plugin in the filesystem #2655 | ||
| * Adopted monorepo structure #2565 |
There was a problem hiding this comment.
This is a pretty big change - perhaps it could be moved up in this list. Although, for users, it is not as that impactful
There was a problem hiding this comment.
Do we need this entry at all? I would skip entries that are about changing the repo structure, build systems, etc.
katinthehatsite
left a comment
There was a problem hiding this comment.
The changes look good to me, I left some minor comments mostly related to grammar
Proposed Changes
Testing Instructions
RELEASE-NOTES.txtfor accuracy and tone consistency with previous releasesPre-merge Checklist