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

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

Merged
merged 1 commit into from
Feb 19, 2020

Conversation

sappelhoff
Copy link
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
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
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