-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adapts procedure registry to depend on the cypher version #331
base: main
Are you sure you want to change the base?
Changes from 1 commit
34f2bb8
1f76037
d1d0a48
d79e282
b9fc356
d653261
33fd518
e58d2e5
44c0b32
6e7e5c6
ff25f9e
fbdf5bb
7cacbcb
9d29a4b
ee324f9
6c98b22
07f20b7
f3b41a1
662b643
cbda4a1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,14 +12,21 @@ import { | |
|
||
import { TextDocument } from 'vscode-languageserver-textdocument'; | ||
|
||
import { syntaxColouringLegend } from '@neo4j-cypher/language-support'; | ||
import { | ||
syntaxColouringLegend, | ||
_internalFeatureFlags, | ||
} from '@neo4j-cypher/language-support'; | ||
import { Neo4jSchemaPoller } from '@neo4j-cypher/schema-poller'; | ||
import { doAutoCompletion } from './autocompletion'; | ||
import { cleanupWorkers, lintDocument } from './linting'; | ||
import { doSignatureHelp } from './signatureHelp'; | ||
import { applySyntaxColouringForDocument } from './syntaxColouring'; | ||
import { Neo4jSettings } from './types'; | ||
|
||
if (process.env.CYPHER_25 === 'true') { | ||
_internalFeatureFlags.cypher25 = true; | ||
} | ||
Comment on lines
+26
to
+28
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is so we can set the feature flag in the language server from the vscode extension |
||
|
||
const connection = createConnection(ProposedFeatures.all); | ||
|
||
// Create a simple text document manager. | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,6 +30,7 @@ export async function activate(context: ExtensionContext) { | |
debug: { | ||
module: debugServer, | ||
transport: TransportKind.ipc, | ||
options: { env: { CYPHER_25: 'true' } }, | ||
Comment on lines
30
to
+33
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This only affects the server version we use in the tests, not the production one. So we'll have |
||
}, | ||
}; | ||
// Options to control the language client | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to pass the feature flags to the lint worker because the
_internalFeatureFlags
variable from the outside lives in another thread