Skip to content
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

Insert new changelog content before the first h2 existing element #91

Merged
merged 3 commits into from Oct 29, 2020

Conversation

Turbo87
Copy link
Contributor

@Turbo87 Turbo87 commented Oct 29, 2020

... or at the start of the file if none is found

Resolves #90

Version headings in lerna-changelog are h2, not h3
... or at the start of the file if none is found
@rwjblue
Copy link
Collaborator

rwjblue commented Oct 29, 2020

This should fix https://github.com/rwjblue/create-rwjblue-release-it-setup/blob/d7b0d4faeb50392a0ef251332797197cb89498ec/__tests__/bin-test.js#L61-L67 then right (by making it not something we want to do, but instead we'd do the opposite and add one)?

@@ -147,7 +147,7 @@ test('it uses the first commit when no tags exist', async (t) => {
value: 'hahahahaah, does not exist',
},
'git rev-list --max-parents=0 HEAD': 'aabc',
[`${LERNA_PATH} --next-version=Unreleased --from=aabc`]: `### Unreleased\n\nThe changelog\n### v1.0.0\n\nThe old changelog`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

ugg didn't realize I had this different than what lerna-changelog did itself 😩

@rwjblue rwjblue merged commit 82d010e into release-it-plugins:master Oct 29, 2020
@rwjblue rwjblue added the enhancement New feature or request label Oct 29, 2020
@Turbo87
Copy link
Contributor Author

Turbo87 commented Oct 29, 2020

by making it not something we want to do, but instead we'd do the opposite and add one)?

I guess, yeah. I hadn't actually seen that test yet 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add new section below h1 element
2 participants