Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Docs: fix a broken link #861

Merged
merged 1 commit into from
Jul 27, 2021
Merged

Docs: fix a broken link #861

merged 1 commit into from
Jul 27, 2021

Conversation

chenxsan
Copy link
Contributor

@eslint-github-bot eslint-github-bot bot added the triage An ESLint team member will look at this issue soon label Jul 24, 2021
@netlify
Copy link

netlify bot commented Jul 24, 2021

✔️ Deploy Preview for eslint ready!

🔨 Explore the source changes: ec6a3a6

🔍 Inspect the deploy log: https://app.netlify.com/sites/eslint/deploys/60fbf2e2572fb30008778600

😎 Browse the preview: https://deploy-preview-861--eslint.netlify.app/docs/rules

Copy link
Contributor

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@snitin315 snitin315 added accepted There is consensus among the team that this change meets the criteria for inclusion documentation Relates to ESLint's documentation and removed triage An ESLint team member will look at this issue soon labels Jul 24, 2021
@bmish
Copy link
Member

bmish commented Jul 24, 2021

If you search for user-guide/command-line-interface#fix), there are a few hundred places with this broken link. Might as well fix them all together?

@chenxsan
Copy link
Contributor Author

@bmish Those we found on this repository are generated automatically. Anyway I've filed another pull request over eslint repository eslint/eslint#14833 to fix them.

@nzakas
Copy link
Member

nzakas commented Jul 27, 2021

Thanks @chenxsan

@nzakas nzakas merged commit 8b99f17 into eslint:master Jul 27, 2021
@chenxsan chenxsan deleted the patch-3 branch July 27, 2021 00:50
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion documentation Relates to ESLint's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants