chore: add test for sql-parser/save-helper/core #24
Conversation
✅ Deploy Preview for tisqleditor-playground ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
baurine
force-pushed
the
chore/add-test-case
branch
from
0999205
to
bfc859f
Compare
baurine
force-pushed
the
chore/add-test-case
branch
from
e67727c
to
232e492
Compare
baurine
changed the title
| @@ -0,0 +1,167 @@ | |||
| import { EditorView } from '@codemirror/view' | |||
There was a problem hiding this comment.
please change the file name to a meaningful one
There was a problem hiding this comment.
Because this extension only has one file, I have no better idea, do you have any recommendation?
| @@ -0,0 +1,95 @@ | |||
| import { EditorView } from '@codemirror/view' | |||
There was a problem hiding this comment.
please change the file name to a meaningful one
There was a problem hiding this comment.
Because this extension only has one file, I have no better idea, do you have any recommendation?
| @@ -37,6 +38,7 @@ | |||
| "typescript": "^5.4.5" | |||
| }, | |||
| "peerDependencies": { | |||
| "@codemirror/lang-sql": "^6.6.4", | |||
There was a problem hiding this comment.
In the test, we need to install sql() extension from lang-sql package with sql-parser extension together, else sql-parser can't work.
| @@ -0,0 +1,95 @@ | |||
| import { EditorView } from '@codemirror/view' | |||
| import { EditorState } from '@codemirror/state' | |||
| import { MySQL, sql } from '@codemirror/lang-sql' | |||
There was a problem hiding this comment.
We use lang-sql here, @sanshuiyijing
No description provided.