Skip to content

Run Prettier on firebird source files#212

Merged
eumalin merged 1 commit into
mainfrom
firebird/prettier-format
May 26, 2026
Merged

Run Prettier on firebird source files#212
eumalin merged 1 commit into
mainfrom
firebird/prettier-format

Conversation

@eumalin
Copy link
Copy Markdown
Contributor

@eumalin eumalin commented May 19, 2026

Runs Prettier over the 18 source files that had inconsistent formatting. The .prettierrc was already in the repo but hadn't been applied to all files. There are only whitespace, quotes, and line wrapping changes.

Here are the Prettier rules for reference:

{
"plugins": ["prettier-plugin-svelte"],
"trailingComma": "es5",
"tabWidth": 2,
"singleQuote": true,
"printWidth": 120
}

Reviewing with "ignore whitespace" enabled (?w=1 on GitHub) may be helpful.

@eumalin eumalin marked this pull request as ready for review May 21, 2026 17:44
@eumalin eumalin requested a review from carylwyatt May 21, 2026 17:44
Copy link
Copy Markdown
Member

@carylwyatt carylwyatt left a comment

Choose a reason for hiding this comment

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

Oh, wow. 🫣
I have prettier installed for VS Code, but it doesn't always format when I save, and I never took the time to figure it out.
Thanks for cleaning this up!

@eumalin
Copy link
Copy Markdown
Contributor Author

eumalin commented May 26, 2026

Oh, wow. 🫣
I have prettier installed for VS Code, but it doesn't always format when I save, and I never took the time to figure it out.
Thanks for cleaning this up!

No worries! Prettier can be finicky with auto-format on save, it usually needs format on save and default formatter set to prettier in VS Code settings. I'll check if I can update the .vscode/settings.json file so that we don't need to do that manually.

@eumalin eumalin merged commit 281acf8 into main May 26, 2026
2 checks passed
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