51fb03b4b1
Fix tests to match code changes
2022-08-10 13:11:34 -04:00
838a2b71ac
Scan logs on change in current query
2022-08-09 18:02:27 -04:00
3c57597a19
Share code for splitting records from pseudo-JSONL
2022-08-05 17:36:45 -04:00
e8d5029912
Merge remote-tracking branch 'origin/main' into dbartol/join-order-temp
2022-08-05 17:34:52 -04:00
cb514f5c78
Pre-cleanup to avoid merge conflicts
2022-08-05 14:59:40 -04:00
57bb8cee41
Update regexes to match new summary text
2022-08-04 16:17:27 -04:00
1219ef4a8c
Remove unnecessary command
2022-08-04 16:17:09 -04:00
677a0f7940
Fix lint
2022-08-04 14:42:47 -04:00
b8cca29eb3
Ensure query history state is persisted after new query is added ( #1451 )
2022-08-04 15:06:47 +01:00
4cbf104bdf
(Minor) Remove outdated comment ( #1453 )
2022-08-04 13:24:48 +00:00
26ccde9e7d
Bump CLI version to 2.10.2 for integration tests
2022-08-03 10:30:55 +01:00
beb5b78b89
Add 50ms wait for flaky telemetry popup test ( #1449 )
2022-08-02 08:24:07 -07:00
c3a21b93c0
Merge pull request #1430 from github/dbartol/goto-ql
...
Initial implementation of sourcemap-based jump-to-QL command
2022-08-01 13:52:06 -04:00
6b9f73e156
Add comment to test data file
2022-08-01 13:19:15 -04:00
6409e09063
Code cleanup
2022-08-01 12:28:35 -04:00
8f5611b074
Move sourcemap tests to `cli-integration`
2022-08-01 12:14:00 -04:00
7f3fcce1ac
Temporarily increase delay for extension activation in test
2022-07-29 13:11:46 -04:00
4bc1d1ed8a
Force activation of extension
2022-07-29 12:44:06 -04:00
02e5b4e830
Fix installation of dependent extensions
2022-07-29 12:03:43 -04:00
538792e8bb
Try installing extension dependencies for `minimal-workspace` tests
2022-07-29 11:35:52 -04:00
56ec970121
Merge branch 'dbartol/goto-ql' of https://github.com/github/vscode-codeql into dbartol/goto-ql
2022-07-29 11:01:07 -04:00
57a04297bd
Only disable specific extensions for `minimal-workspace` tests
2022-07-29 11:01:02 -04:00
59f1e4e90a
Update extensions/ql-vscode/src/pure/log-summary-parser.ts
2022-07-28 22:31:18 -04:00
7c1fce3319
Merge remote-tracking branch 'origin/main' into dbartol/goto-ql
2022-07-28 22:29:45 -04:00
476ea7aef0
Integration test
2022-07-28 22:20:22 -04:00
0c654c4320
Merge pull request #1444 from github/elenatanasoiu/fix-bugs
...
Don't show parentheses when results are not yet fetched in Query History
2022-07-26 10:33:32 +01:00
895ac6ae26
Squash extra whitespace for Query History labels
...
We'd like to remove duplicate whitespace in these labels in order
to make it less likely that we introduce extra space.
We initially also tried trimming whitespaces at the start and end
of these labels but that had no effect.
2022-07-26 09:49:27 +01:00
52484f1211
Don't show parentheses when results are not yet fetched
...
We missed a place where we needed to check if results are present
before attempting to show them.
Let's also add tests for this.
2022-07-26 09:47:55 +01:00
cba188b4db
Use named arguments for mock function
...
We'd like to be able to add tests for when the result count exists and
when it's missing.
Let's change the createMockRemoteQueryInfo method so that we can pass
in parameters by name, e.g.
```
createMockRemoteQueryInfo(undefined, 2)
```
becomes
```
createMockRemoteQueryInfo({ repositoryCount: 2 }
```
2022-07-26 09:46:10 +01:00
123b1fc085
Clarify title description
...
To make it clear it's referring to a `user-specified` label, not that the user is not specified.
2022-07-25 17:16:36 +01:00
833f8e06ca
Add a tree viewer UI for the evaluator logs ( #1433 )
...
Co-authored-by: Aditya Sharad <6874315+adityasharad@users.noreply.github.com>
2022-07-22 12:01:39 +02:00
747049ed1b
Merge pull request #1435 from github/dependabot/npm_and_yarn/extensions/ql-vscode/yargs-parser-20.2.4
...
Bump yargs-parser from 5.0.0-security.0 to 20.2.4 in /extensions/ql-vscode
2022-07-20 08:47:15 -07:00
d62e9181f2
Merge pull request #1436 from github/dependabot/npm_and_yarn/extensions/ql-vscode/semver-regex-and-husky-3.1.4
...
Bump semver-regex and husky in /extensions/ql-vscode
2022-07-20 08:45:04 -07:00
e4d1f4e73e
Fix newline handling for cross-platform logs
...
We were splitting JSONL records based on the current OS newline sequence. In order to handle reading of logs from the opposite OS, I've switched our split to handle both flavors of line ending. This originally showed up as log parser unit tests failing on Windows (the checked-in log used Unix line endings), but could affect real world usage as well.
2022-07-20 11:21:53 -04:00
c1922126d3
Bump terser from 5.14.1 to 5.14.2 in /extensions/ql-vscode
...
Bumps [terser](https://github.com/terser/terser ) from 5.14.1 to 5.14.2.
- [Release notes](https://github.com/terser/terser/releases )
- [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md )
- [Commits](https://github.com/terser/terser/commits )
---
updated-dependencies:
- dependency-name: terser
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-07-20 12:03:15 +01:00
d2ebb3d20a
Bump version to v1.6.10
2022-07-20 11:57:41 +01:00
72858e341a
Bump CLI version to 2.10.1 for integration tests ( #1442 )
2022-07-20 11:55:43 +02:00
4499773f6f
Merge pull request #1440 from github/v1.6.9
...
v1.6.9
2022-07-20 10:16:21 +01:00
1d3b0e0ca9
v1.6.9
2022-07-20 10:01:12 +01:00
98e503c768
Merge pull request #1438 from github/shati-patel/gist-description
...
MRVA: Fix Gist description when repository count is undefined
2022-07-20 09:46:22 +01:00
62c3974d35
Check for `undefined`, `null` or zero repositories
...
`undefined`, `null` and 0 will evaluate to `false` so if we only want to
display the repository count when these values are not present we can
check for a truthy value:
```
query.repositoryCount ? `(${pluralize(...)})` : '';
```
instead of checking explicitly:
```
query.repositoryCount !== undefined && query.repositoryCount !== null && query.repositoryCount != 0 ? `(${pluralize(...)})` : '';
```
2022-07-20 09:30:54 +01:00
40e0027074
Fix newline handling for cross-platform logs
...
We were splitting JSONL records based on the current OS newline sequence. In order to handle reading of logs from the opposite OS, I've switched our split to handle both flavors of line ending. This originally showed up as log parser unit tests failing on Windows (the checked-in log used Unix line endings), but could affect real world usage as well.
2022-07-19 17:29:33 -04:00
ab1c2e0a0d
Explicitly check for `undefined`
2022-07-19 20:00:10 +01:00
d918c41197
Fix Gist description when repository count is undefined
2022-07-19 18:25:25 +01:00
84048ccac1
Merge remote-tracking branch 'origin/main' into dbartol/goto-ql
2022-07-19 09:39:51 -04:00
cbb09da0d0
Bump semver-regex and husky in /extensions/ql-vscode
...
Bumps [semver-regex](https://github.com/sindresorhus/semver-regex ) and [husky](https://github.com/typicode/husky ). These dependencies needed to be updated together.
Updates `semver-regex` from 2.0.0 to 3.1.4
- [Release notes](https://github.com/sindresorhus/semver-regex/releases )
- [Commits](https://github.com/sindresorhus/semver-regex/compare/v2.0.0...v3.1.4 )
Updates `husky` from 4.2.5 to 4.3.8
- [Release notes](https://github.com/typicode/husky/releases )
- [Commits](https://github.com/typicode/husky/compare/v4.2.5...v4.3.8 )
---
updated-dependencies:
- dependency-name: semver-regex
dependency-type: indirect
- dependency-name: husky
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-07-19 12:51:24 +00:00
c8d3428f21
Bump yargs-parser in /extensions/ql-vscode
...
Bumps [yargs-parser](https://github.com/yargs/yargs-parser ) from 5.0.0-security.0 to 20.2.4.
- [Release notes](https://github.com/yargs/yargs-parser/releases )
- [Changelog](https://github.com/yargs/yargs-parser/blob/main/CHANGELOG.md )
- [Commits](https://github.com/yargs/yargs-parser/commits/v20.2.4 )
---
updated-dependencies:
- dependency-name: yargs-parser
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-07-19 12:51:22 +00:00
2cf5b39cfe
Merge pull request #1432 from github/charisk-elena/result-count-on-history-labels
...
Add result count to remote queries in Query History
2022-07-19 13:50:22 +01:00
13921bf8a2
Extract sum method for adding up repo results
...
When a queryResult is created, it comes with an array for AnalysisSummaries.
There is one summary per repository.
We've had to calculate the total number of results for all summaries in multiple
places, so let's extract a method for this as well.
2022-07-19 13:26:56 +01:00
12a97ecba2
Shorten param forwarding for repositoryCount
2022-07-19 13:26:54 +01:00
Dave Bartolomeo
Charis Kyriakou
Shati Patel
Angela P Wen
Elena Tanasoiu
Andrew Eisenberg
dependabot[bot]
elenatanasoiu