Add support for aliasing languages like C/C++, Java/Kotlin, and JS/TS - #1883
henrymercer merged 4 commits into
Conversation
Sorry, something went wrong.
angelapwen
left a comment
There was a problem hiding this comment.
Approving to unblock, but have a couple of comments. Please feel free to get approval from another 🇪🇺 person once addressed!
Sorry, something went wrong.
| /** | ||
| * Versions 2.14.4+ of the CodeQL CLI support language aliasing. | ||
| */ | ||
| export const CODEQL_VERSION_LANGUAGE_ALIASING = "2.14.4"; |
There was a problem hiding this comment.
Hm.. should we bump this to 2.14.5 given its rollout on dotcom was stopped and bumped directly to 2.14.5?
Sorry, something went wrong.
There was a problem hiding this comment.
2.14.5 is identical to 2.14.4 apart from a JavaScript fix, so I think it's fine to use 2.14.4 here. As you say though, it doesn't affect that many customers.
Sorry, something went wrong.
| ## [UNRELEASED] | ||
|
|
||
| - Update default CodeQL bundle version to 2.14.5. [#1882](https://github.com/github/codeql-action/pull/1882) | ||
| - Enable the following language aliases when using CodeQL 2.14.4 and later: `c-cpp` for C/C++ analysis, `java-kotlin` for Java/Kotlin analysis, and `javascript-typescript` for JavaScript/TypeScript analysis. [#1883](https://github.com/github/codeql-action/pull/1883) |
There was a problem hiding this comment.
Same comment as above on whether we want to move this to 2.14.5.
Sorry, something went wrong.
This PR adds support for the c-cpp, java-kotlin, and javascript-typescript aliases for C/C++, Java/Kotlin, and JavaScript/TypeScript analysis. To ensure a consistent experience across all CodeQL products, we implemented these aliases in the CLI, so this PR is really just exposing that functionality.
Merge / deployment checklist