Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

feat(vscode): enable formatting for JSX and TSX files #2795

Merged
merged 3 commits into from
Jun 29, 2022

Conversation

ematipico
Copy link
Contributor

Summary

This PR enables formatting of jsx and tsx files inside the VS Code extension.

Test Plan

I tried locally by installing it on my VS Code. The user has to manually enable Rome as default formatter for these files

@ematipico ematipico requested review from leops and yassere June 29, 2022 10:47
@cloudflare-pages
Copy link

cloudflare-pages bot commented Jun 29, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4373fae
Status: ✅  Deploy successful!
Preview URL: https://2abe2692.tools-8rn.pages.dev
Branch Preview URL: https://feature-enable-jsx-vscode.tools-8rn.pages.dev

View logs

@ematipico ematipico changed the title fix(rome_js_formatter): indentation before arrow fn is JSX feat(vscode): enable formatting for JSX/TSX Jun 29, 2022
@ematipico ematipico changed the title feat(vscode): enable formatting for JSX/TSX feat(vscode): enable formatting for JSX and TSX files Jun 29, 2022
@@ -3,11 +3,13 @@
"publisher": "rome",
"displayName": "Rome",
"description": "Rome LSP VS Code Extension",
"version": "0.8.0",
"version": "0.9.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this change shouldn't have been committed since bumping the version number will trigger a release workflow, and the 0.9.0 version number is reserved for pre-releases of the extension

Copy link
Contributor Author

@ematipico ematipico Jun 29, 2022

Choose a reason for hiding this comment

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

Shoot! Good catch! Yes, it should NOT be here

Base automatically changed from fix/indentation-before-arrow-fn to main June 29, 2022 17:54
@ematipico ematipico merged commit 5df71ca into main Jun 29, 2022
@ematipico ematipico deleted the feature/enable-jsx-vscode branch June 29, 2022 17:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants