Skip to content

Commit

Permalink
Fix bundle version
Browse files Browse the repository at this point in the history
It's the whole tag, we don't want to remove the `codeql-bundle-` prefix.
  • Loading branch information
henrymercer committed Apr 18, 2023
1 parent 9a866ed commit 1c2f282
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions .github/actions/update-bundle/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ interface Defaults {
priorCliVersion: string;
}

const CODEQL_BUNDLE_PREFIX = 'codeql-bundle-';

function getCodeQLCliVersionForRelease(release): string {
// We do not currently tag CodeQL bundles based on the CLI version they contain.
// Instead, we use a marker file `cli-version-<version>.txt` to record the CLI version.
Expand All @@ -37,7 +35,7 @@ function getCodeQLCliVersionForRelease(release): string {

async function getBundleInfoFromRelease(release): Promise<BundleInfo> {
return {
bundleVersion: release.tag_name.substring(CODEQL_BUNDLE_PREFIX.length),
bundleVersion: release.tag_name,
cliVersion: getCodeQLCliVersionForRelease(release)
};
}
Expand Down

0 comments on commit 1c2f282

Please sign in to comment.