-
Notifications
You must be signed in to change notification settings - Fork 0
chore(release): align package versions to 0.1.0 #105
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
Conversation
cb0080f to
2cdecf0
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Greptile Summary
|
| Filename | Overview |
|---|---|
| packages/daemon/CHANGELOG.md | Contains duplicate 0.1.0 changelog entries causing version history confusion |
| packages/config/CHANGELOG.md | Has duplicate 0.1.0 entries with conflicting version information |
| packages/testing/CHANGELOG.md | Contains duplicate 0.1.0 version entries with different formats and dates |
Confidence score: 3/5
- This PR contains formatting issues that could cause confusion in version tracking and break semantic versioning tooling
- Score lowered due to duplicate changelog entries in multiple packages (daemon, config, testing, file-ops, logging, state, cli, ui) where automated changeset entries conflict with existing manual entries
- Pay close attention to all CHANGELOG.md files that now contain duplicate version entries for 0.1.0
Sequence Diagram
sequenceDiagram
participant Developer as "Developer"
participant Release as "Release Process"
participant Packages as "Package Files"
participant Changelogs as "Changelog Files"
Developer->>Release: "Initiate version alignment to 0.1.0"
Release->>Packages: "Update version in package.json files"
Release->>Changelogs: "Add 0.1.0 entries to CHANGELOG.md files"
Release->>Packages: "Update workspace dependencies to 0.1.0"
Release->>Developer: "Version alignment complete"
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.
Additional Comments (6)
-
packages/file-ops/CHANGELOG.md, line 20 (link)logic: Duplicate version heading - there are now two
## 0.1.0entries which will cause confusion in the changelog -
packages/cli/CHANGELOG.md, line 15-18 (link)style: The changelog header now appears after the new version entry instead of at the top - this breaks the standard changelog format where the header should come first
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
-
packages/config/CHANGELOG.md, line 3-20 (link)logic: Duplicate version entries for 0.1.0 - the new entry at line 3 conflicts with existing entry at line 20
-
packages/ui/CHANGELOG.md, line 20 (link)logic: Duplicate 0.1.0section - there are now two entries for version 0.1.0 (lines 3 and 20)
-
packages/state/CHANGELOG.md, line 3-20 (link)logic: There are now two
0.1.0entries in this changelog - one at line 3 (new) and another at line 20 (existing dated entry). This creates duplicate version entries which violates changelog conventions.Should the existing dated entry at line 20 be removed or merged with the new entry?
-
packages/logging/CHANGELOG.md, line 3-19 (link)logic: Duplicate version 0.1.0 entries with different dates and formats may confuse users about which features belong to which release
Should the changeset-generated entry (lines 3-12) replace the manually written one (lines 19-36), or are these meant to represent different releases?
25 files reviewed, 8 comments
2cdecf0 to
d848054
Compare
75f449f to
ffc2821
Compare
|
Resolved duplicate 0.1.0 changelog entries by merging version-alignment notes into the existing 0.1.0 sections (added Changed/Dependencies where appropriate). Restacked and resubmitted. |
d848054 to
6ff08a4
Compare
6ff08a4 to
b024a0b
Compare
|
Changelog duplicates are resolved: packages/daemon/CHANGELOG.md and packages/testing/CHANGELOG.md now each have a single 0.1.0 entry. |
b024a0b to
4ba4d42
Compare

Summary
Changes
Testing