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

docs: fix the broken link in analyzer/index.mdx #2400

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

ganta
Copy link
Contributor

@ganta ganta commented Apr 10, 2024

Summary

The link to the "Import Sorting" page in docs/{ja,zh-cn,pt-br}/analyzer/index.mdx is broken.

Test Plan

Ensure that the link is working.

@ganta ganta requested a review from a team as a code owner April 10, 2024 16:01
@github-actions github-actions bot added the A-Website Area: website label Apr 10, 2024
Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit 9ebe4ee
🔍 Latest deploy log https://app.netlify.com/sites/biomejs/deploys/6616baa3e761200008290aa0
😎 Deploy Preview https://deploy-preview-2400--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 93 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@Conaclos
Copy link
Member

Conaclos commented Apr 10, 2024

Thanks! I think this affects also other languages. Would you like to fix them as well?

@ganta
Copy link
Contributor Author

ganta commented Apr 10, 2024

Since I am here, I will fix the other languages as well 👌🏼

@ganta ganta marked this pull request as draft April 10, 2024 16:09
@ganta ganta force-pushed the fix-broken-link-in-analyzer-index-ja branch from 8097f04 to 9ebe4ee Compare April 10, 2024 16:13
@ganta ganta changed the title docs(ja): fix the broken link in analyzer/index.mdx docs: fix the broken link in analyzer/index.mdx Apr 10, 2024
@ganta ganta marked this pull request as ready for review April 10, 2024 16:14
@ganta
Copy link
Contributor Author

ganta commented Apr 10, 2024

I have fixed it in all languages 🙌🏼

@Conaclos Conaclos merged commit 08ab901 into biomejs:main Apr 10, 2024
8 checks passed
@Conaclos
Copy link
Member

Great! Thanks!

@ganta ganta deleted the fix-broken-link-in-analyzer-index-ja branch April 10, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Website Area: website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants