8000 docs: fix more problems with markdown link check by MalteHerrmann · Pull Request #575 · evmos/evmos · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

docs: fix more problems with markdown link check #575

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

Merged
merged 16 commits into from
May 9, 2022
Merged

docs: fix more problems with markdown link check #575

merged 16 commits into from
May 9, 2022

Conversation

MalteHerrmann
Copy link
Contributor

Description

I have continued to fix problems with the markdown link checker introduced in #548. Now there should only be actually dead links marked, where I could not find the corresponding resources.

Closes: #XXX


All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

PR review checkboxes:

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • included the correct type prefix in the PR title
  • targeted the correct branch (see PR Targeting)
  • provided a link in the PR description to the relevant issue or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all required CI checks have passed

Code maintenance:

I have...

  • written unit and integration tests
  • added relevant godoc and code comments.
  • updated relevant documentation (docs/) or specification (x/<module>/spec/)

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code

@fedekunze
Copy link
Collaborator

thanks @MalteHerrmann, can you fix the few errors from the md link?

@MalteHerrmann
Copy link
Contributor Author

The references that now still fail the tests point to files that don't exist in the repo anymore.
The files are:

  • docs/architecture/PROCESS.md
  • docs/validators/quickstart/interact_node.md --> should this be changed to for example docs/developers/localnet/single_node.md?
  • docs/validators/governance/proposals/proposal-template.md

I could offer to delete the references from the docs as you probably had a good reason to remove these files from the docs in the first place?

@fedekunze
Copy link
Collaborator

@MalteHerrmann souds good, can you remove the first and second links? For docs/validators/quickstart/interact_node.md we can change it to point to the testnet explorer.

@MalteHerrmann
Copy link
Contributor Author

Alright, that's done. ✌️

@fedekunze fedekunze enabled auto-merge (squash) May 9, 2022 13:16
@fedekunze fedekunze merged commit a4e95a1 into evmos:main May 9, 2022
@MalteHerrmann MalteHerrmann deleted the docs-fix-broken-md-links branch May 9, 2022 14:20
loredanacirstea added a commit that referenced this pull request May 11, 2022
* main: (24 commits)
  docs: update hardhat guide (#594)
  audit(erc20): completeness (#592)
  fix: readd erc20 migration logic (#588)
  release: v3.0.1 changelog (#584)
  release: set v4 upgrade heights (#581)
  docs: fix more problems with markdown link check (#575)
  bump(ethermint): v0.15.0 (#580)
  ci: update mergify config (#579)
  release: v4 upgrade logic (#555)
  docs: upgrade deps (#578)
  docs: upgrade documentation (#572)
  Correct testnet endpoint URLs (#577)
  build(deps): bump docker/setup-qemu-action from 1 to 2 (#566)
  build(deps): bump docker/setup-buildx-action from 1 to 2 (#567)
  ci(fuzz): add fuzz function for x/incentives/types.SplitGasMeterKey (#573)
  tests(erc20): add integration tests (#571)
  build(deps): bump docker/login-action from 1 to 2 (#565)
  tests(erc20): refactor keeper tests (#561)
  Use universal cosmos gitpod image as an editing environment (#564)
  build(deps): bump docker/build-push-action from 2 to 3 (#568)
  ...
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
0