Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 22, 2025

Description

Accidentally deleted this script.


Important

Restores script to update or reformat changelog sections for specific versions using environment variables.

  • Script Restoration:
    • Restores overwrite_changeset_changelog.py to update or reformat CHANGELOG.md.
  • Functionality:
    • Uses environment variables CHANGELOG_PATH, VERSION, PREV_VERSION, and NEW_CONTENT.
    • Updates specified version section in changelog, replacing or reformatting content.
    • Ensures version numbers are wrapped in square brackets.
  • Misc:
    • Adds .changeset/cold-donuts-clean.md for changelog entry.

This description was created by Ellipsis for 32e0c72. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested a review from mrubens as a code owner May 22, 2025 19:01
@cte cte merged commit 45e2263 into main May 22, 2025
13 checks passed
@cte cte deleted the cte/fix-changelog branch May 22, 2025 19:01
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
print(f"latest version: {VERSION}")
print(f"prev_version: {PREV_VERSION}")

notes_start_index = changelog_text.find(version_pattern) + len(version_pattern)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No check if '## {VERSION}\n' exists. If not found, find() returns -1, causing an incorrect slice. Add validation.

+ parsed_lines
+ changelog_text[notes_end_index:]
)
updated_changelog = updated_changelog.replace(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Global replacement of ## {VERSION} with ## [{VERSION}] can inadvertently alter other parts of the file. Limit the replacement to the header.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants