Commits
Commits on Jun 10, 2021
-
Remove unnecessary type assertions
Andrew Eisenberg committedJun 10, 2021 Copy the full SHA 20aafcdView commit details
Commits on Jun 9, 2021
-
Cleans up the output logs even more. These entries don't add any more information since everything is grouped now.
Andrew Eisenberg committedJun 9, 2021 Copy the full SHA 23cdd6bView commit details -
Add pack download to its own log group
Also, make the baseline count message less awkward sounding.
Andrew Eisenberg committedJun 9, 2021 Copy the full SHA a2e96a4View commit details -
Use nullish conversion for packs
Slightly simplifies the `parsePacks` function.
Andrew Eisenberg committedJun 9, 2021 Copy the full SHA 96e7de3View commit details -
Extract a `runTool` function that captures the stdout and returns it. A small refactoring that reduces copied code.
Andrew Eisenberg committedJun 9, 2021 Copy the full SHA 49b2220View commit details -
Merge remote-tracking branch 'upstream/main' into aeisenberg/pack-run
Andrew Eisenberg committedJun 9, 2021 Copy the full SHA 82388fdView commit details -
Add
--finalize-dataset
todatabase finalize
callEdoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021 Copy the full SHA babcc1bView commit details -
Add new log grouping for DB cleanup
Edoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021 Copy the full SHA 06df98aView commit details -
Add intepret-results timings to status reports
Edoardo Pirovano authored and Edoardo Pirovano committedJun 9, 2021 Copy the full SHA 0ea20c5View commit details
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 Copy the full SHA d42f654View commit details -
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 Copy the full SHA 1cc5f1dView commit details -
Replace analyze with run-queries and interpret-results
Edoardo Pirovano authored and Edoardo Pirovano committedJun 8, 2021 Copy the full SHA 2cc885dView commit details
Commits on Jun 7, 2021
-
Some refactoring in fingerprint computation
Edoardo Pirovano authored and Edoardo Pirovano committedJun 7, 2021 Copy the full SHA c095005View commit details -
Fix out of memory in hash computation
Edoardo Pirovano authored and Edoardo Pirovano committedJun 7, 2021 Copy the full SHA 02e8dcfView commit details -
Merge branch 'main' into retrying-fix
Chris Gavin authored and GitHub committedJun 7, 2021 Copy the full SHA 48efe6eView commit details -
Chris Gavin committed
Jun 7, 2021 Copy the full SHA f8c65b7View commit details
Commits on Jun 6, 2021
-
Andrew Eisenberg committed
Jun 6, 2021 Copy the full SHA d87945eView commit details -
Add
--sarif-group-rules-by-pack
flagEdoardo Pirovano authored and Edoardo Pirovano committedJun 6, 2021 Copy the full SHA 1644adeView commit details
Commits on Jun 4, 2021
-
Use strings instead. They are easier to serialize and deserialize.
Andrew Eisenberg committedJun 4, 2021 Copy the full SHA 06687e9View commit details -
Fix logic for calculating if there are queries to run
During the analyze phase.
Andrew Eisenberg committedJun 4, 2021 Copy the full SHA 9b5753aView commit details -
Cleanup CodeQL DBs and output their location for later steps
Edoardo Pirovano authored and Edoardo Pirovano committedJun 4, 2021 Copy the full SHA ddb83a4View commit details -
Add better comments and error messages for pack-related changes
Andrew Eisenberg committedJun 4, 2021 Copy the full SHA 6cee818View commit details -
Chris Gavin committed
Jun 4, 2021 Copy the full SHA 41dff7fView commit details
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 Copy the full SHA 86a804fView commit details -
Cleaning up comments around env vars
Also, move `getTemporaryDirectory` back to `actions-util`.
Andrew Eisenberg committedJun 3, 2021 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 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 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 Copy the full SHA 3708898View commit details -
Merge branch 'main' into update-supported-enterprise-server-versions
Andrew Eisenberg authored and GitHub committedJun 2, 2021 Copy the full SHA 739e14dView commit details -
Update supported GitHub Enterprise Server versions.
GitHub committedJun 2, 2021 Copy the full SHA 2fee124View commit details
Commits on Jun 1, 2021
-
Use commander preAction hook for setMode
Hooks are new to commander v8. We can use hooks to ensure that `setMode` is called before every command is invoked.
Andrew Eisenberg committedJun 1, 2021 Copy the full SHA 539d968View commit details
Commits on May 31, 2021
-
Use the version from package.json in the runner
Andrew Eisenberg committedMay 31, 2021 Copy the full SHA 1a4cdd3View commit details -
Send the version and mode with the user agent
This commit changes the way the action determines if running in action or runner mode. There is now an environment variable that is set at the beginning of the process and elsewhere in the process, we can check to see if the variable is set.
Andrew Eisenberg committedMay 31, 2021 Copy the full SHA 4758879View commit details
Commits on May 28, 2021
-
Analyze: Log analysis summaries for custom queries
Henry Mercer committedMay 28, 2021 Copy the full SHA 2338fe5View commit details