-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
docs: fix broken link to Prism includeLangs.js default list of languages #8904
Conversation
✅ [V2]Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the deploy preview of this PR
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reporting.
Not 100% sure it's the correct replacement link
Also we need to fix older Docusaurus docs versions too ;)
By default, Docusaurus comes with a subset of [commonly used languages](https://github.com/FormidableLabs/prism-react-renderer/blob/master/src/vendor/prism/includeLangs.js). | ||
By default, Docusaurus comes with a subset of [commonly used languages](https://github.com/FormidableLabs/prism-react-renderer/blob/master/packages/generate-prism-languages/index.ts). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure it's the proper replacement link (nor if it actually still exists). Waiting for an answer in FormidableLabs/prism-react-renderer#190
fix #8910 |
Pre-flight checklist
Motivation
I found a broken link, and I am making a pull request to fix it.
Test Plan
Just click on the new link.
Here is the broken link and here is the new corresponding link .
Test links
Not needed as this is just a docs change.
Related issues/PRs
None