Skip to content

Clean up semicolons after interface blocks#209

Merged
JasonShin merged 6 commits intoJasonShin:mainfrom
Leo5878:remove-interface-semicolons
Jun 15, 2025
Merged

Clean up semicolons after interface blocks#209
JasonShin merged 6 commits intoJasonShin:mainfrom
Leo5878:remove-interface-semicolons

Conversation

@Leo5878
Copy link
Contributor

@Leo5878 Leo5878 commented Jun 14, 2025

This pull request removes unnecessary semicolons after interface declarations across the codebase.

Semicolons after interface blocks are syntactically allowed in TypeScript but not required. They are often considered noise and are discouraged by popular style guides (e.g., TypeScript Handbook, Airbnb). This change improves stylistic consistency without affecting functionality.

@JasonShin
Copy link
Owner

JasonShin commented Jun 15, 2025

Thanks again, @Leo5878!

My thinking was that even if sqlx-ts generates types in a specific style, consumers would likely apply their own Prettier or ESLint configurations to format the generated code anyway. That said, it's still a good idea for the default output to follow standard formatting conventions as a baseline.

This is now released as v0.23.0 https://github.com/JasonShin/sqlx-ts/releases/tag/v0.23.0

@JasonShin JasonShin merged commit b9c55b7 into JasonShin:main Jun 15, 2025
18 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