Skip to content

docs(oxfmt): Include links for the VS Code extension in the formatter FAQ#658

Merged
Boshen merged 1 commit intomainfrom
connorshea-patch-2
Dec 2, 2025
Merged

docs(oxfmt): Include links for the VS Code extension in the formatter FAQ#658
Boshen merged 1 commit intomainfrom
connorshea-patch-2

Conversation

@connorshea
Copy link
Member

@connorshea connorshea commented Nov 26, 2025

Lead with the VS Code extension in the answer, since that's the best and main officially-supported route for many users. Also rephrase the wording slightly.

We should maybe also note that the user will need to enable the formatter via a setting in the VS Code extension? Unless we intend to remove that step in the very near future.

… FAQ

Lead with the VS Code extension since that's the best route for many users. Also rephrase the wording slightly.

Signed-off-by: Connor Shea <connor.james.shea@gmail.com>
@netlify
Copy link

netlify bot commented Nov 26, 2025

Deploy Preview for oxc-project ready!

Name Link
🔨 Latest commit 41be488
🔍 Latest deploy log https://app.netlify.com/projects/oxc-project/deploys/6926a1a73e79290008e88c77
😎 Deploy Preview https://deploy-preview-658--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.

@connorshea connorshea requested a review from leaysgur November 26, 2025 06:46
Copy link
Member

@leaysgur leaysgur left a comment

Choose a reason for hiding this comment

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

I think the content is good. 👌🏻

However, in fact, as of this very moment, I understand that the VSCode extension does not yet support oxfmt. 🙈

oxc-project/oxc#15837

@connorshea
Copy link
Member Author

I think the content is good. 👌🏻

However, in fact, as of this very moment, I understand that the VSCode extension does not yet support oxfmt. 🙈

oxc-project/oxc#15837

Good point! We can probably hold off on this change for a few days if we want to.

@Boshen Boshen merged commit ad1ff62 into main Dec 2, 2025
7 checks passed
@Boshen Boshen deleted the connorshea-patch-2 branch December 2, 2025 02:06
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.

3 participants