Commits
82388fd94a
Name already in use
Commits on Jun 9, 2021
-
Merge remote-tracking branch 'upstream/main' into aeisenberg/pack-run
Andrew Eisenberg committedJun 9, 2021 -
Add
--finalize-dataset
todatabase finalize
callEdoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021 -
Add new log grouping for DB cleanup
Edoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021 -
Add intepret-results timings to status reports
Edoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021
Commits on Jun 8, 2021
-
Add an integration test for packaging
Uses two pre-existing packages to run some simple queries on a javascript database.
Andrew Eisenberg committedJun 8, 2021 -
Packaging: Address review comments
1. Better malformed data guard for PackDownloadOutput 2. Fix Packs type 3. Remove TODO in init-action
Andrew Eisenberg committedJun 8, 2021 -
Replace analyze with run-queries and interpret-results
Edoardo Pirovano authored and Edoardo Pirovano committedJun 8, 2021
Commits on Jun 7, 2021
-
Some refactoring in fingerprint computation
Edoardo Pirovano authored and Edoardo Pirovano committedJun 7, 2021 -
Merge pull request #554 from github/mergeback/v1.0.1-to-main-ece2addc
Mergeback v1.0.1 refs/heads/v1 into main
Andrew Eisenberg authored and GitHub committedJun 7, 2021 -
Merge branch 'main' into mergeback/v1.0.1-to-main-ece2addc
Andrew Eisenberg authored and GitHub committedJun 7, 2021 -
github-actions[bot] committed
Jun 7, 2021 -
Update changelog and version after v1.0.1
github-actions[bot] committedJun 7, 2021 -
Merge pull request #553 from github/aeisenberg/no-quotes
Remove quotes around `$DRAFT` argument
Andrew Eisenberg authored and GitHub committedJun 7, 2021 -
Remove quotes around
$DRAFT
argumentgh will complain if there is an empty `""`, which is the case if `$DRAFT` is unset.
Andrew Eisenberg committedJun 7, 2021 -
Merge pull request #549 from github/update-v1.0.1-242fd828
Merge main into v1
Chris Gavin authored and GitHub committedJun 7, 2021 -
Chris Gavin committed
Jun 7, 2021 -
Fix out of memory in hash computation
Edoardo Pirovano authored and Edoardo Pirovano committedJun 7, 2021 -
github-actions[bot] committed
Jun 7, 2021 -
Merge pull request #547 from github/retrying-fix
Add back retrying.
Chris Gavin authored and GitHub committedJun 7, 2021 -
Merge branch 'main' into retrying-fix
Chris Gavin authored and GitHub committedJun 7, 2021 -
Chris Gavin committed
Jun 7, 2021
Commits on Jun 6, 2021
-
Andrew Eisenberg committed
Jun 6, 2021 -
Add
--sarif-group-rules-by-pack
flagEdoardo Pirovano authored and Edoardo Pirovano committedJun 6, 2021
Commits on Jun 4, 2021
-
Use strings instead. They are easier to serialize and deserialize.
Andrew Eisenberg committedJun 4, 2021 -
Fix logic for calculating if there are queries to run
During the analyze phase.
Andrew Eisenberg committedJun 4, 2021 -
Cleanup CodeQL DBs and output their location for later steps
Edoardo Pirovano authored and Edoardo Pirovano committedJun 4, 2021 -
Add better comments and error messages for pack-related changes
Andrew Eisenberg committedJun 4, 2021 -
Chris Gavin committed
Jun 4, 2021
Commits on Jun 3, 2021
-
Allow the codeql-action to run packages
This commit adds a `packs` option to the codeql-config.yml file. Users can specify a list of ql packs to include in the analysis. For a single language analysis, the packs property looks like this: ```yaml packs: - pack-scope/pack-name1@1.2.3 - pack-scope/pack-name2 # no explicit version means download the latest ``` For multi-language analysis, you must key the packs block by lanaguage: ```yaml packs: cpp: - pack-scope/pack-name1@1.2.3 - pack-scope/pack-name2 java: - pack-scope/pack-name3@1.2.3 - pack-scope/pack-name4 ``` This implementation adds a new analysis run (alongside custom and builtin runs). The unit tests indicate that the correct commands are being run, but I have not actually tried this with a real CLI. Also, convert `instanceof Array` to `Array.isArray` since that is sightly better in some situations. See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/isArray#instanceof_vs_isarray
Andrew Eisenberg committedJun 3, 2021 -
Merge pull request #539 from github/aeisenberg/env-vars
Add environment variables to signal feature and version to the CLI
Andrew Eisenberg authored and GitHub committedJun 3, 2021 -
Cleaning up comments around env vars
Also, move `getTemporaryDirectory` back to `actions-util`.
Andrew Eisenberg committedJun 3, 2021 Unverified
No user is associated with the committer email.Copy the full SHA f60ef17View commit details
Commits on Jun 2, 2021
-
Fix Unguarded actions library use code scanning error
Must access the env var directly in order to avoid recursive calls to `getRequiredEnvParam`.
Andrew Eisenberg committedJun 2, 2021 Copy the full SHA cc0733fView commit details -
Merge branch 'main' into aeisenberg/env-vars
Andrew Eisenberg committedJun 2, 2021 Unverified
No user is associated with the committer email.Copy the full SHA 082575fView commit details -
Remove local environment running
This is a functionality that never worked perfectly and hasn't been used for a while. This allows developers to run the action on their local machine, but the run was always flaky and never 100% mirrored what was happening on the actions runner.
Andrew Eisenberg committedJun 2, 2021 Unverified
No user is associated with the committer email.Copy the full SHA 2c2ebdcView commit details -
Add environment variables to signal feature and version to the CLI
This PR ensures environment variables are set before any invocation of the CLI. Here is a list of vars that are set: https://github.com/github/codeql-coreql-team/issues/1124#issuecomment-852463521 This ensures the CLI knows the features and versions of the containing actions/runner. Additionally: - Fix the user agent so that it more closely aligns with user agent spec - Refactor environment variable initialization so that it all happens in one place and call. - Move Mode, getRequiredEnvParam, setMode, getMode out of actions-util and into util. actions-util is meant for utils only called by the action, not the runner. The `prepareLocalRunEnvironment()` method is most likely deprecated and should be removed. I originally added it because I had a way of working where I would run the action from my local machine to test out changes, but this was always a little flaky. So, I no longer use this way of working. I will probably remove it soon.
Andrew Eisenberg committedJun 2, 2021 Unverified
No user is associated with the committer email.Copy the full SHA 3708898View commit details