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

feat(changelog): add PR links in markdown #282

Conversation

wyardley
Copy link
Contributor

@wyardley wyardley commented Dec 5, 2020

GitHub release notes automagically link to the PR, but changelog markdown doesn't. Replace (#123) at the end of a message with a markdown link.

@wyardley wyardley force-pushed the wyardley-changelog-replace-pr-links branch from 52ced32 to 9d0d988 Compare December 5, 2020 07:06
@wyardley wyardley changed the title feat(changelog): add PR links in changelog feat(changelog): add PR links in markdown Dec 5, 2020
@wyardley
Copy link
Contributor Author

wyardley commented Dec 5, 2020

Open to suggestions on style / formatting here. I decided to do the f string for the table formatting in two lines, and to put the spaces at the beginning of the next line for the tests, but not super married to either way of doing it.

Similarly, could easily make the regex part little terser or change stuff there if it would be preferred.

@wyardley wyardley force-pushed the wyardley-changelog-replace-pr-links branch from 9d0d988 to a3baf4f Compare December 5, 2020 07:33
@wyardley wyardley force-pushed the wyardley-changelog-replace-pr-links branch 2 times, most recently from 241c1e0 to 7a48056 Compare December 5, 2020 18:34
GitHub release notes automagically link to the PR, but changelog
markdown doesn't. Replace (python-semantic-release#123) at the end of a message with a markdown
link.
@wyardley wyardley force-pushed the wyardley-changelog-replace-pr-links branch from 7a48056 to b45e159 Compare December 6, 2020 19:19
@wyardley wyardley requested a review from danth December 7, 2020 19:34
@wyardley
Copy link
Contributor Author

wyardley commented Dec 9, 2020

Also, we may want to (fairly soon) update the gitlab ones to follow the same hostname as for the API, assuming that's the correct behavior, but that would probably go along with the changes I mentioned in #277

@danth danth merged commit 0448f6c into python-semantic-release:master Dec 12, 2020
@wyardley wyardley deleted the wyardley-changelog-replace-pr-links branch July 22, 2021 18:29
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.

None yet

2 participants