Skip to content

Use quotes for db and table names that contain non-alphanumeric chars#18

Merged
dawsontoth merged 2 commits into
mainfrom
dts-quote-db-table-names
May 20, 2026
Merged

Use quotes for db and table names that contain non-alphanumeric chars#18
dawsontoth merged 2 commits into
mainfrom
dts-quote-db-table-names

Conversation

@cb1kenobi
Copy link
Copy Markdown
Member

If you have a database or table name that contains a dash, it causes invalid syntax in the generated globalTypes.d.ts file.

@cb1kenobi cb1kenobi requested a review from a team as a code owner May 18, 2026 16:56
@cb1kenobi cb1kenobi marked this pull request as draft May 18, 2026 17:02
@cb1kenobi cb1kenobi marked this pull request as ready for review May 18, 2026 19:14
Copy link
Copy Markdown
Contributor

@dawsontoth dawsontoth left a comment

Choose a reason for hiding this comment

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

I like it! Mind fixing the commit title prefix for conventional commits?

@cb1kenobi cb1kenobi force-pushed the dts-quote-db-table-names branch 2 times, most recently from 051fb2f to e0c8b67 Compare May 20, 2026 19:58
@cb1kenobi cb1kenobi force-pushed the dts-quote-db-table-names branch from e0c8b67 to a0e4774 Compare May 20, 2026 20:00
@cb1kenobi
Copy link
Copy Markdown
Member Author

@dawsontoth done! When can we get rid of conventional commits?

@dawsontoth
Copy link
Copy Markdown
Contributor

Why? It drives the semantic versioning here.

@dawsontoth dawsontoth merged commit 3fd76a0 into main May 20, 2026
9 checks passed
@dawsontoth dawsontoth deleted the dts-quote-db-table-names branch May 20, 2026 21:07
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants