Skip to content

BREAKING: Format changelog using Prettier #116

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

Merged
merged 4 commits into from
Oct 19, 2023
Merged

Conversation

Mrtenz
Copy link
Member

@Mrtenz Mrtenz commented Sep 28, 2023

MetaMask/auto-changelog#155 adds a method to format changelogs using Prettier. I've enabled this option here.

This is a breaking change, since it will change the default formatting of the changelog in new release PRs. Packages updating to the next version of this package should simultaneously enable the --prettier flag on auto-changelog.

Breaking changes

@socket-security
Copy link

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@metamask/auto-changelog 3.1.0...3.3.0 None +1/-0 11.4 MB metamaskbot
prettier 2.2.1...2.8.8 environment +0/-0 11.2 MB prettier-bot

Copy link

@MajorLift MajorLift left a comment

Choose a reason for hiding this comment

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

Could we add a BREAKING changelog entry in the PR description (and eventual commit message) detailing the required build script changes, and any other considerations to introducing the updates made in this PR?

Otherwise looks good!

Copy link

@MajorLift MajorLift left a comment

Choose a reason for hiding this comment

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

Looks good!

@Mrtenz Mrtenz merged commit 0e7eb9d into main Oct 19, 2023
@Mrtenz Mrtenz deleted the mrtenz/prettier-formatter branch October 19, 2023 08:50
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.

2 participants