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

doc: remove hoe-markdown #3033

Merged
merged 2 commits into from
Nov 19, 2023
Merged

doc: remove hoe-markdown #3033

merged 2 commits into from
Nov 19, 2023

Conversation

flavorjones
Copy link
Member

What problem is this PR intended to solve?

The CHANGELOG.md file has had hyperlinked issues, PRs, and usernames since 0ac9033 / 2020-02-02 so that, when rendered on nokogiri.org, users could easily get to deeper information.

The downside, though, is that when the changelog snippets are copied to release notes, usernames aren't added to the "contributors" list for the release; and the release notes consumed by tools like dependabot and libraries.io don't really need the hyperlinks and it's sometimes just messy-looking.

So this PR removes the markdown linkification, and updates the CHANGELOG to just have bare references like #1234 and @flavorjones.

Separately, I've added hoe-markdown to nokogiri.org in sparklemotion/nokogiri.org@4fa918d so the site will still have everything linkified.

Have you included adequate test coverage?

unneeded.

Does this change affect the behavior of either the C or the Java implementations?

N/A

We'll rely on github to linkify this doc properly in the repo, and
move markdownification into nokogiri.org for rendering the site.
@flavorjones flavorjones merged commit 6f0849e into main Nov 19, 2023
126 checks passed
@flavorjones flavorjones deleted the flavorjones-remove-hoe-markdown branch November 19, 2023 19:53
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.

1 participant