Skip to content

[INFRA] fix heading of auto changelog to be a markdown header#417

Merged
sappelhoff merged 1 commit intobids-standard:masterfrom
sappelhoff:headerch
Feb 19, 2020
Merged

[INFRA] fix heading of auto changelog to be a markdown header#417
sappelhoff merged 1 commit intobids-standard:masterfrom
sappelhoff:headerch

Conversation

@sappelhoff
Copy link
Copy Markdown
Member

I am changing the header-label [LABEL] parameter to include a # sign, so that hopefully the header of our changelog file will be rendered as an actual header.

see: #400 (comment)

@sappelhoff
Copy link
Copy Markdown
Member Author

sappelhoff commented Feb 19, 2020

I am merging this now, because it is a minor infrastructure change that may need iterations.

Edit: I falsely thought that I could see the changes of this addition upon merging this PR ... however, the changes will only become apparent upon our next release (i.e., upon the next PR that contains REL: in its description)

However, I did check the change with a local installation of github-changelog-generator and it worked as expected.

@sappelhoff sappelhoff merged commit 3e5b1e8 into bids-standard:master Feb 19, 2020
@sappelhoff sappelhoff deleted the headerch branch February 19, 2020 09:50
@franklin-feingold
Copy link
Copy Markdown
Collaborator

it works on merged PRs - when a REL is noticed it does not make a new changelog commit

appears this works - https://github.com/bids-standard/bids-specification/blob/master/src/CHANGES.md

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