Skip to content

feat(changelog): Auto generate changelog when empty - #287

Merged
BYK merged 2 commits into
masterfrom
byk/feat/auto-changelog-items
Sep 15, 2021
Merged

feat(changelog): Auto generate changelog when empty#287
BYK merged 2 commits into
masterfrom
byk/feat/auto-changelog-items

Conversation

@BYK

@BYK BYK commented Sep 14, 2021

Copy link
Copy Markdown
Member

No description provided.

@BYK BYK added this to the Automated Changelog Generation milestone Sep 14, 2021
@BYK
BYK enabled auto-merge (squash) September 15, 2021 07:14

@iker-barriocanal iker-barriocanal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we add tests for the PR ID format in the commit messages?

Comment thread src/utils/git.ts
Comment thread src/commands/prepare.ts
Comment thread src/commands/prepare.ts
Comment thread src/utils/changes.ts
@BYK

BYK commented Sep 15, 2021

Copy link
Copy Markdown
Member Author

Should we add tests for the PR ID format in the commit messages?

Thought about that but I think a better test would be to test the new changelog generation logic: make sure we have a PR id at the end, if not fallback to short SHA etc. For that I need to construct a sample/fake repo in tests.

That said since this feature is ephemeral and will evolve very quickly to use GitHub milestones, I think delaying tests for that next step makes more sense.

@iker-barriocanal iker-barriocanal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

馃殌

Comment thread src/commands/prepare.ts
@BYK
BYK merged commit 5f13741 into master Sep 15, 2021
@BYK
BYK deleted the byk/feat/auto-changelog-items branch September 15, 2021 12:14
Comment thread src/utils/git.ts
Comment thread src/commands/prepare.ts
Comment thread CHANGELOG.md
Comment thread src/utils/changes.ts
BYK added a commit that referenced this pull request Sep 17, 2021
BYK added a commit that referenced this pull request Sep 17, 2021
* ref: Add comments for changelog extraction

Follow up to #287

Co-authored-by: Chad Whitacre <chadwhitacre@sentry.io>
BYK added a commit that referenced this pull request Sep 20, 2021
This patch implements the GitHub Milestones based automatic
changelog generation. It builds up on #285 and #287 and ties
them with GitHub milestone data to generate a changelog that
is composed of logical sections with references to relevant
PRs.

Any remaining items are put under "Various fixes & improvements"
section as an itemized list.
BYK added a commit that referenced this pull request Sep 23, 2021
This patch implements the GitHub Milestones based automatic
changelog generation. It builds up on #285 and #287 and ties
them with GitHub milestone data to generate a changelog that
is composed of logical sections with references to relevant
PRs.

Any remaining items are put under "Various fixes & improvements"
section as an itemized list.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants