Skip to content

fix 1.9.0-beta1 changelog formatting #8941

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 3 commits into from
Oct 14, 2020
Merged

Conversation

rboyer
Copy link
Member

@rboyer rboyer commented Oct 13, 2020

I think there's something subtly off with the automagic changelog formatter that causes this.

@rboyer rboyer requested review from hanshasselberg, mikemorris, a team and preetapan and removed request for mikemorris October 13, 2020 20:37
@mikemorris
Copy link
Contributor

Is that one entry the only instance that needs to be fixed? Trying to figure out why that would be the case...

@rboyer
Copy link
Member Author

rboyer commented Oct 13, 2020

Yes. I found this one because make dev invoked make changelogfmt which "fixed" this wrongly and dirtied my working tree. I fixed it the right way and then the make target was still happy.

Side note: maybe that make target needs to be smarter, also we might want to run it in CI to fail the build if the changelog isn't clean already.

@mikemorris
Copy link
Contributor

mikemorris commented Oct 13, 2020

Oh, nothing wrong with the auto-formatter here, just a reference to the original issue that was inadvertently added in the changelog entry in #8842, will push up a fix.

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