fix: soft disable mismatch indicator for sqljs driver #148
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since this driver does not report the column type, the type should be treated as
undefined
, which causes the type analysis system to be disabled for this driver.Since you mentioned that you’re implementing MySQL support, it’s not necessary to approve this PR right now. When you're ready, let me know, and I’ll do a rebase and adjust whatever is needed.
As for this PR, in short, I just removed the internal indication that forced the
TEXT
type for all columns when usingsqljs
to be undefined. This way, the mismatch indicator system will be disabled.I also migrated the code I had created in
global.css
to pure Tailwind, which I believe is ideal. I also fixed a typo.