Skip to content

Comments

docs(linter): fix error in curly "all" example.#642

Closed
jacobcarpenter wants to merge 1 commit intooxc-project:mainfrom
jacobcarpenter:patch-1
Closed

docs(linter): fix error in curly "all" example.#642
jacobcarpenter wants to merge 1 commit intooxc-project:mainfrom
jacobcarpenter:patch-1

Conversation

@jacobcarpenter
Copy link

@jacobcarpenter jacobcarpenter commented Nov 17, 2025

The current docs' example of correct code for curly "all" has an error. "all" requires braces around the statement bodies of all control statements.

Signed-off-by: Jacob Carpenter <github@jacobcarpenter.com>
@netlify
Copy link

netlify bot commented Nov 17, 2025

Deploy Preview for oxc-project ready!

Name Link
🔨 Latest commit 145f74f
🔍 Latest deploy log https://app.netlify.com/projects/oxc-project/deploys/691b654ff74dd60008d50e89
😎 Deploy Preview https://deploy-preview-642--oxc-project.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 project configuration.

@camc314
Copy link
Contributor

camc314 commented Nov 18, 2025

Thank you. These docs are auto-generated. I have updated them here oxc-project/oxc#15801

@jacobcarpenter
Copy link
Author

Awesome; thanks!

I had just followed the "Suggest changes to this page" link in the footer without checking on whether that was the authoritative source.

Now that you've made that change, should this PR be closed?

@camc314 camc314 closed this Nov 19, 2025
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