chore(replace): run prettier on changelog

[tjenkinson]

Rollup Plugin Name: replace

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

Description

Prettier is failing because of this missing newline, and therefore blocking the checks on #921

A fix might be needed in the publish script to ensure a new line at the end of the file.

[github-actions]

Your PR title doesn't match the required format. The title should be in the conventional commit (https://www.conventionalcommits.org/en/v1.0.0-beta.4/) format. e.g.

chore(plugin-name): add pr title workflow

[shellscape]

I'm actually more in favor of removing the check. It's incredibly annoying for automation. Prettier will be run on any files modified in the release commit, including Changelings, and automatically committed.

[tjenkinson]

I think it's useful to have the catch-all, but we could exclude those auto generated files from prettier in the .prettierignore?

[guybedford]

Having these files in prettierignore makes sense to me.

Also moving prettier to the release process only sounds like a potential way around these issues in future as well.

But given this is holding up all PRs right now, I'm all for merging the quickest fix here for now.

[tjenkinson]

Having a new line at the end of files also seems to be a good practice though (github also shows warnings), although I must admit I'm not sure why

[shellscape]

@guybedford when you're merging PRs, please make sure you're double checking the commit message that Github suggests to use. If the PR was created with a title that wasn't conventional commit style, and we update that title later, the suggested commit is typically the original non-compliant title.