Skip to content

Commit

Permalink
Add run number in artifact name
Browse files Browse the repository at this point in the history
  • Loading branch information
jprinet committed Sep 17, 2024
1 parent 60deab9 commit f621826
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 3 deletions.
3 changes: 2 additions & 1 deletion build-scan-shared/src/setup/post.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {DefaultArtifactClient} from '@actions/artifact'

import * as input from './input'
import * as commonBuildTool from '../buildTool/common'
import * as githubUtils from '../utils/github'
import * as summary from '../summary/dump'

export async function post(buildTool: commonBuildTool.BuildTool): Promise<void> {
Expand Down Expand Up @@ -48,5 +49,5 @@ async function uploadArtifacts(
function getArtifactName(buildScanArtifactName: string): string {
return `${buildScanArtifactName}-${input.getWorkflowName().replaceAll(' ', '-')}-${input
.getJobName()
.replaceAll(' ', '-')}`
.replaceAll(' ', '-')}-${githubUtils.getRunNumber()}`
}
4 changes: 4 additions & 0 deletions build-scan-shared/src/utils/github.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,10 @@ function getRunActorForTest(): string {
return github.context.actor
}

export function getRunNumber(): number {
return github.context.runNumber
}

function getListArtifactsOptions(): any {
return isUnderTest() ? getListArtifactsOptionsForTest() : getListArtifactsOptionsForWorkflowRun()
}
Expand Down
4 changes: 4 additions & 0 deletions dist/maven-publish-build-scan/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -135459,6 +135459,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
Object.defineProperty(exports, "__esModule", ({ value: true }));
exports.getOctokit = getOctokit;
exports.isPublicationAllowed = isPublicationAllowed;
exports.getRunNumber = getRunNumber;
exports.getArtifactIdsForWorkflowRun = getArtifactIdsForWorkflowRun;
exports.downloadBuildScanData = downloadBuildScanData;
exports.downloadArtifact = downloadArtifact;
Expand Down Expand Up @@ -135497,6 +135498,9 @@ function getRunActorForWorkflowRun() {
function getRunActorForTest() {
return github.context.actor;
}
function getRunNumber() {
return github.context.runNumber;
}
function getListArtifactsOptions() {
return isUnderTest() ? getListArtifactsOptionsForTest() : getListArtifactsOptionsForWorkflowRun();
}
Expand Down
7 changes: 6 additions & 1 deletion dist/maven-setup/post/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -135280,6 +135280,7 @@ const core = __importStar(__nccwpck_require__(42186));
const glob = __importStar(__nccwpck_require__(28090));
const artifact_1 = __nccwpck_require__(79450);
const input = __importStar(__nccwpck_require__(24758));
const githubUtils = __importStar(__nccwpck_require__(54377));
const summary = __importStar(__nccwpck_require__(14277));
async function post(buildTool) {
await uploadBuildScanDataFiles(buildTool);
Expand Down Expand Up @@ -135313,7 +135314,7 @@ async function uploadArtifacts(files, buildScanDataFolder, buildScanArtifactName
function getArtifactName(buildScanArtifactName) {
return `${buildScanArtifactName}-${input.getWorkflowName().replaceAll(' ', '-')}-${input
.getJobName()
.replaceAll(' ', '-')}`;
.replaceAll(' ', '-')}-${githubUtils.getRunNumber()}`;
}


Expand Down Expand Up @@ -135478,6 +135479,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
Object.defineProperty(exports, "__esModule", ({ value: true }));
exports.getOctokit = getOctokit;
exports.isPublicationAllowed = isPublicationAllowed;
exports.getRunNumber = getRunNumber;
exports.getArtifactIdsForWorkflowRun = getArtifactIdsForWorkflowRun;
exports.downloadBuildScanData = downloadBuildScanData;
exports.downloadArtifact = downloadArtifact;
Expand Down Expand Up @@ -135516,6 +135518,9 @@ function getRunActorForWorkflowRun() {
function getRunActorForTest() {
return github.context.actor;
}
function getRunNumber() {
return github.context.runNumber;
}
function getListArtifactsOptions() {
return isUnderTest() ? getListArtifactsOptionsForTest() : getListArtifactsOptionsForWorkflowRun();
}
Expand Down
3 changes: 2 additions & 1 deletion release/changelog.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
- Fixes a bug where setting the develocity-token-expiry parameter had no effect
- Update Develocity Maven extension to 1.22.1
- Add run attempt number to the artifact name containing the build scan data and metadata
- Update Develocity Maven extension to 1.22.1

0 comments on commit f621826

Please sign in to comment.