Skip to content

Conversation

CamWass
Copy link
Contributor

@CamWass CamWass commented Apr 25, 2025

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[X] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

This PR removes a double space in the docs that was caused by incorrect usage of the npx_tabs macro.
Screenshot 2025-04-25 at 4 10 03 pm

Is there anything you'd like reviewers to focus on?

@CamWass CamWass requested a review from a team as a code owner April 25, 2025 06:12
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Apr 25, 2025
Copy link

linux-foundation-easycla bot commented Apr 25, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@eslint-github-bot eslint-github-bot bot added the documentation Relates to ESLint's documentation label Apr 25, 2025
Copy link

netlify bot commented Apr 25, 2025

Deploy Preview for docs-eslint ready!

Name Link
🔨 Latest commit d71cf36
🔍 Latest deploy log https://app.netlify.com/sites/docs-eslint/deploys/680b2ca87127030008f0ea1e
😎 Deploy Preview https://deploy-preview-19657--docs-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Comment on lines 26 to 29
{{ npx_tabs({
package: null,
args: ["@eslint/migrate-config", ".eslintrc.json"]
package: "@eslint/migrate-config",
args: [".eslintrc.json"]
}) }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the actual change, everything else is just auto-formatting

Copy link
Member

@mdjermanovic mdjermanovic 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!

@mdjermanovic mdjermanovic merged commit c0721a7 into eslint:main Apr 25, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Relates to ESLint's documentation
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

2 participants