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

Fix code formatting in Preview Web UI #24691

Merged
merged 1 commit into from
Jan 12, 2025
Merged

Conversation

koszti
Copy link
Member

@koszti koszti commented Jan 12, 2025

Description

The TypeScript code in the preview web UI is currently not properly formatted due to an incorrect Prettier configuration, which does not include recursive formatting for .ts and .tsx files. This PR resolves the issue by updating the Prettier configuration and reformatting all files to the correct format.

No functional changes are included.

Additional context and related issues

The fix involves updating the Prettier target in package.json (located here) by changing the pattern from ./src/*.{ts,tsx} to ./src/**/*.{ts,tsx} in order to include all recursive TypeScript files.

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label Jan 12, 2025
@github-actions github-actions bot added the ui Web UI label Jan 12, 2025
@wendigo wendigo merged commit 990ab98 into trinodb:master Jan 12, 2025
94 checks passed
@github-actions github-actions bot added this to the 469 milestone Jan 12, 2025
@koszti koszti deleted the node-prettier-fix branch January 12, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants