Граф коммитов

3940 Коммитов

Автор SHA1 Сообщение Дата
Robert fc3bce8d41 Convert extensions/ql-vscode/src/variant-analysis/export-results.ts to call typed commands 2023-03-23 16:03:20 +00:00
Robert b7edf1efec Convert extensions/ql-vscode/src/query-server/queryserver-client.ts to call typed commands 2023-03-23 15:58:22 +00:00
Charis Kyriakou 8641b83506 Rename SetupServerApi to SetupServer 2023-03-23 15:56:53 +00:00
Koen Vlaswinkel 2a0fd46952 Merge branch 'koesie10/react-18' into koesie10/react-strict-mode 2023-03-23 16:54:49 +01:00
Koen Vlaswinkel d4464c8662 Merge remote-tracking branch 'origin/main' into koesie10/react-18 2023-03-23 16:53:17 +01:00
Robert f84bf2dba0 Convert extensions/ql-vscode/src/mocks/vscode-mock-gh-api-server.ts to call typed commands 2023-03-23 15:53:12 +00:00
Robert 6b3a235a77 Convert extensions/ql-vscode/src/log-insights/summary-language-support.ts to call typed commands 2023-03-23 15:53:12 +00:00
Elena Tanasoiu 07c79058af
Merge branch 'main' into elena/disable-local-retry 2023-03-23 15:36:33 +00:00
Elena Tanasoiu c882a9fc14
Only retry tests in CI
We introduced this change to help with reducing flakiness in CI [1].

This has a slightly different effect locally, where every failed test
will output three times.

This in turn makes it harder to read, especially when you have multiple
failing tests.

Since the original intent for this behaviour was to be used in CI, I'm
proposing we disable it when the CI env variable isn't set.

I've opted to set it for all jobs involving tests, just for consistency.
I'm happy to limit it to just the places where it's required.

[1]: https://github.com/github/vscode-codeql/pull/2059
2023-03-23 15:28:51 +00:00
Robert d56a986cc5
Merge pull request #2222 from github/robertbrignull/use_app_commands_2
Convert call sites to use typed commands (part 2): QueryHistoryManager
2023-03-23 15:07:31 +00:00
Koen Vlaswinkel 16b9abed43
Merge pull request #2220 from github/koesie10/update-unique-command-use-typed-commands
Add support for typed commands to CodeQL query
2023-03-23 16:00:33 +01:00
Charis Kyriakou 7837f3077c
Merge pull request #2221 from github/dependabot/npm_and_yarn/extensions/ql-vscode/octokit/plugin-throttling-5.0.1
Bump @octokit/plugin-throttling from 4.3.2 to 5.0.1 in /extensions/ql-vscode
2023-03-23 14:25:55 +00:00
Robert 180b84b3da Use correct variable name 2023-03-23 14:25:15 +00:00
dependabot[bot] 53067f8249
Bump msw from 0.49.0 to 1.2.0 in /extensions/ql-vscode
Bumps [msw](https://github.com/mswjs/msw) from 0.49.0 to 1.2.0.
- [Release notes](https://github.com/mswjs/msw/releases)
- [Changelog](https://github.com/mswjs/msw/blob/main/CHANGELOG.md)
- [Commits](https://github.com/mswjs/msw/compare/v0.49.0...v1.2.0)

---
updated-dependencies:
- dependency-name: msw
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-03-23 14:02:27 +00:00
dependabot[bot] faec3cda5d
Bump fs-extra and @types/fs-extra in /extensions/ql-vscode
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) and [@types/fs-extra](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/fs-extra). These dependencies needed to be updated together.

Updates `fs-extra` from 10.0.1 to 11.1.1
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](https://github.com/jprichardson/node-fs-extra/compare/10.0.1...11.1.1)

Updates `@types/fs-extra` from 9.0.6 to 11.0.1
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/fs-extra)

---
updated-dependencies:
- dependency-name: fs-extra
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: "@types/fs-extra"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-03-23 14:00:56 +00:00
dependabot[bot] de0a8bf94d
Bump @octokit/plugin-throttling in /extensions/ql-vscode
Bumps [@octokit/plugin-throttling](https://github.com/octokit/plugin-throttling.js) from 4.3.2 to 5.0.1.
- [Release notes](https://github.com/octokit/plugin-throttling.js/releases)
- [Commits](https://github.com/octokit/plugin-throttling.js/compare/v4.3.2...v5.0.1)

---
updated-dependencies:
- dependency-name: "@octokit/plugin-throttling"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-03-23 13:59:20 +00:00
Robert c84c69f529 Convert extensions/ql-vscode/src/vscode-utils/external-files.ts to call typed commands 2023-03-23 13:49:11 +00:00
Robert a9e495118c Convert extensions/ql-vscode/src/query-history/query-history-manager.ts to call typed commands 2023-03-23 13:49:11 +00:00
Robert 2e7952cb37 Pull tryOpenExternalFile out to a separate file so it can be more easily tested 2023-03-23 13:49:11 +00:00
Robert 6bcfdda0a9
Merge pull request #2217 from github/robertbrignull/use_app_commands_1
Convert call sites to use typed commands (part 1)
2023-03-23 13:18:49 +00:00
Robert e0dd8c7392 Add comment about codeQLDatabases.focus 2023-03-23 13:04:30 +00:00
Koen Vlaswinkel 164536338b
Merge pull request #2219 from github/koesie10/fix-command-runner-references
Remove remaining references to `commandRunner`
2023-03-23 14:03:39 +01:00
Koen Vlaswinkel f57bfb6875 Add support for typed commands to CodeQL query
This adds support for detecting the `CommandManager.execute` method in
the unique command use query.

This may not be the best way to implement this. There's a method
`hasUnderlyingType` on `this.getReceiver().getType()`, but I couldn't
really figure out how to get it recognize `CommandManager`. It might be
possible if we can construct the type of `CommandManager`, but this will
probably include the filepath to the `CommandManager` class, which might
not neccessarily be something we want: moving the `CommandManager` class
should not require updating the query. I'm very happy to hear other
suggestions.
2023-03-23 13:55:52 +01:00
Koen Vlaswinkel d0c4e5b8c5 Remove remaining references to `commandRunner` 2023-03-23 13:30:06 +01:00
Koen Vlaswinkel 96c9a4aca7
Merge pull request #2215 from github/koesie10/rename-command-runner
Rename and move `commandRunner`
2023-03-23 13:20:49 +01:00
Robert 14dc391229 Convert extensions/ql-vscode/src/legacy-query-server/queryserver-client.ts to call typed commands 2023-03-23 11:42:14 +00:00
Robert d7e061e159 Convert extensions/ql-vscode/src/databases/ui/db-panel.ts to use typed commands 2023-03-23 11:42:14 +00:00
Robert 2a2cb26597 Convert extensions/ql-vscode/src/test-ui.ts to call typed commands 2023-03-23 11:42:14 +00:00
Robert ff418b5487 Convert extensions/ql-vscode/src/query-editor.ts to call typed commands 2023-03-23 11:42:14 +00:00
Robert 8b9003e845 Convert extensions/ql-vscode/src/local-databases.ts to call typed commands 2023-03-23 11:42:12 +00:00
Robert c394b7c4f0 Convert extensions/ql-vscode/src/helpers.ts to call typed commands 2023-03-23 11:42:09 +00:00
Robert ddf5b3aac2 Convert extensions/ql-vscode/src/databaseFetcher.ts to call typed commands 2023-03-23 11:42:06 +00:00
github-actions[bot] 75634b3654
Bump version to v1.8.2 (#2218) 2023-03-23 11:39:09 +00:00
Shati Patel 4c7c1cb0e6
v1.8.1 (#2216) 2023-03-23 11:00:29 +00:00
Robert 13c3c8efe4 Convert extensions/ql-vscode/src/cli.ts to call typed commands 2023-03-23 10:49:48 +00:00
Robert a3c4c8fb61
Merge pull request #2212 from github/robertbrignull/app_executeCommand
Replace app.executeCommand with app.commands.execute
2023-03-23 10:28:36 +00:00
Koen Vlaswinkel 73b0a0565f
Merge pull request #2214 from github/koesie10/remove-command-runner-with-progress
Remove `commandRunnerWithProgress`
2023-03-23 11:26:20 +01:00
Koen Vlaswinkel aa268cfc5f Rename and move `commandRunner`
The `commandRunner` name doesn't really make sense since it doesn't
"run" a command, but rather registers a command. This renames it
to `registerCommandWithErrorHandling` and moves it to the
`common/vscode` directory.
2023-03-23 11:14:44 +01:00
Robert 8fbde9fba1 Limit setContext keys 2023-03-23 10:12:20 +00:00
Koen Vlaswinkel aca0489fdc Remove `commandRunnerWithProgress`
The `commandRunnerWithProgress` is unused because all commands have been
converted to using `commandRunner` in combination with `withProgress`.
2023-03-23 11:09:32 +01:00
Koen Vlaswinkel dae472ca05
Merge pull request #2213 from github/koesie10/check-for-updates-typed-command
Add `codeQL.checkForUpdatesToCLI` type for command
2023-03-23 10:49:18 +01:00
Koen Vlaswinkel 20f85f2b81
Merge pull request #2210 from github/koesie10/base-typed-commands
Convert some base commands to typed commands
2023-03-23 10:03:53 +01:00
Koen Vlaswinkel 81cce9fa8b Add `codeQL.checkForUpdatesToCLI` type for command
The `codeQL.checkForUpdatesToCLI` command is registered pre-activation,
and we don't really want to create the command manager before
activation, so this will just add the correct type without registering
it using the command manager.
2023-03-23 09:58:27 +01:00
Koen Vlaswinkel a27ca2e177 Merge remote-tracking branch 'origin/main' into koesie10/base-typed-commands 2023-03-23 09:51:24 +01:00
Robert 3e9c2c85d3 Remove app.executeCommand 2023-03-22 17:13:12 +00:00
Robert 297fa2ebd3
Merge pull request #2207 from github/robertbrignull/extension_commands
Start using app.commands.execute for all commands called from extension.ts
2023-03-22 16:59:53 +00:00
Shati Patel 844e58a1b1
Tidy up recent changelog entries (#2211) 2023-03-22 16:54:17 +00:00
Robert 9948b93b76 Merge branch 'main' into robertbrignull/extension_commands 2023-03-22 16:35:56 +00:00
Koen Vlaswinkel e1d5a4ddaa Merge remote-tracking branch 'origin/main' into koesie10/base-typed-commands 2023-03-22 17:22:26 +01:00
Koen Vlaswinkel 7059f46141
Merge pull request #2209 from github/koesie10/query-editor-typed-commands
Convert query editing commands to typed commands
2023-03-22 17:21:31 +01:00