Skip to content
This repository was archived by the owner on Oct 22, 2026. It is now read-only.

Parse changelog entry adjustments - #248

Merged
gauthierpetetin merged 15 commits into
parse-changelog-entryfrom
parse-changelog-entry-adjustments
Sep 17, 2025
Merged

gauthierpetetin merged 15 commits into
parse-changelog-entryfrom
parse-changelog-entry-adjustments

Conversation

@gauthierpetetin

@gauthierpetetin gauthierpetetin commented Sep 8, 2025 •

Copy link
Copy Markdown
Contributor

This PR includes a few adjustments required in @HowardBraham 's PR:

  • The strings included in keywordsToIndicateExcluded need to be set to lowercase.
  • The changelog generation should be idempotent

@gauthierpetetin gauthierpetetin self-assigned this Sep 8, 2025
@gauthierpetetin
gauthierpetetin requested a review from a team as a code owner September 8, 2025 17:26
@gauthierpetetin
gauthierpetetin marked this pull request as draft September 8, 2025 17:27
cursor[bot]

This comment was marked as outdated.

@gauthierpetetin
gauthierpetetin force-pushed the parse-changelog-entry-adjustments branch 2 times, most recently from ecb2463 to 91720df Compare September 10, 2025 09:37
@gauthierpetetin
gauthierpetetin force-pushed the parse-changelog-entry-adjustments branch from 91720df to 3298e52 Compare September 10, 2025 10:25
@gauthierpetetin
gauthierpetetin marked this pull request as ready for review September 10, 2025 10:25
Comment thread src/update-changelog.ts Resolved
Signed-off-by: Pavel Dvorkin <pavel.dvorkin@consensys.net>
@XxdpavelxX
XxdpavelxX force-pushed the parse-changelog-entry-adjustments branch from f3bec04 to 3a32aab Compare September 11, 2025 13:37
cursor[bot]

This comment was marked as outdated.

@gauthierpetetin
gauthierpetetin merged commit 3c3cb18 into parse-changelog-entry Sep 17, 2025
@gauthierpetetin
gauthierpetetin deleted the parse-changelog-entry-adjustments branch September 17, 2025 07:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants