Skip to content

Added vscode.q to the release artifact#751

Open
bjeffery-kx wants to merge 11 commits into
devfrom
q-api-benj-2
Open

Added vscode.q to the release artifact#751
bjeffery-kx wants to merge 11 commits into
devfrom
q-api-benj-2

Conversation

@bjeffery-kx

Copy link
Copy Markdown
Collaborator

Changes introduced by this PR

  • vscode.q should now be included in the release artifacts
  • Removed normalizeQSQLQuery around the query wrapper for Python code, as it seems to have been included accidentally
  • Removed an unused parameter
  • Cleaned up comments

@bjeffery-kx bjeffery-kx requested a review from ecmel May 25, 2026 19:42
@bjeffery-kx bjeffery-kx self-assigned this May 25, 2026
ecmel
ecmel previously approved these changes May 28, 2026
@ecmel ecmel marked this pull request as draft May 29, 2026 13:49
@ecmel ecmel marked this pull request as ready for review June 4, 2026 15:22
…, rather than just the error message.

This is especially important as the extension doesn't format errors differently than results, so you need the callstack to indicate when something is an error.
@kx-sonarqube

kx-sonarqube Bot commented Jun 4, 2026

Copy link
Copy Markdown

@bjeffery-kx

Copy link
Copy Markdown
Collaborator Author

When this is ready, https://gitlab.com/kxdev/documentation/vscode-docs/-/blob/main/docs/guides/execute-code.md?ref_type=heads needs to be updated to say how to get vscode.q

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