Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Don't re-retrieve GitHub version when already have it
Sam Partington committed Jan 6, 2021

Unverified

No user is associated with the committer email.
1 parent 023add5 commit c7275a7
Showing 6 changed files with 6 additions and 10 deletions.
3 changes: 1 addition & 2 deletions lib/analyze-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/analyze-action.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 1 addition & 2 deletions lib/runner.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/runner.js.map

Large diffs are not rendered by default.

3 changes: 1 addition & 2 deletions src/analyze-action.ts
@@ -84,7 +84,6 @@ async function run() {
);

if (actionsUtil.getRequiredInput("upload") === "true") {
const gitHubVersion = await util.getGitHubVersion(apiDetails);
const uploadStats = await upload_lib.uploadFromActions(
outputDir,
parseRepositoryNwo(
@@ -97,7 +96,7 @@ async function run() {
actionsUtil.getWorkflowRunID(),
actionsUtil.getRequiredInput("checkout_path"),
actionsUtil.getRequiredInput("matrix"),
gitHubVersion,
config.gitHubVersion,
apiDetails,
logger
);
3 changes: 1 addition & 2 deletions src/runner.ts
@@ -392,14 +392,13 @@ program
return;
}

const gitHubVersion = await getGitHubVersion(apiDetails);
await upload_lib.uploadFromRunner(
outputDir,
parseRepositoryNwo(cmd.repository),
cmd.commit,
parseRef(cmd.ref),
cmd.checkoutPath || process.cwd(),
gitHubVersion,
config.gitHubVersion,
apiDetails,
logger
);

0 comments on commit c7275a7

Please sign in to comment.