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

ci: Remove Markdown link checker #455

Merged
merged 4 commits into from
Mar 6, 2023
Merged

ci: Remove Markdown link checker #455

merged 4 commits into from
Mar 6, 2023

Conversation

thanethomson
Copy link
Contributor

Link checking will soon happen from the CometBFT docs repo, as per cometbft/cometbft-docs#54.

Already, the Jekyll build restructures the documentation, resulting in broken links in Markdown that will be correctly rendered only once deployed to the docs site (e.g. the /spec directory is moved into /docs/spec, and the rpc/openapi/* contents are moved into /docs/rpc/ during the build for each version of the docs).


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@thanethomson thanethomson added the ci Continuous integration-related label Mar 4, 2023
@thanethomson thanethomson requested a review from a team as a code owner March 4, 2023 13:36
@mergify mergify bot merged commit a2066cf into main Mar 6, 2023
@mergify mergify bot deleted the thane/ci/rm-linkcheck branch March 6, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge ci Continuous integration-related
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants