diff --git a/src/persistence/persistence.ts b/src/persistence/persistence.ts index 6f93a77c..ecbee079 100644 --- a/src/persistence/persistence.ts +++ b/src/persistence/persistence.ts @@ -35,31 +35,30 @@ export async function createFile(contentBinary): Promise { }) } -export async function updateFile( - sha: string, - claFileContent, - reactedCommitters: ReactedCommitterMap -): Promise { - const octokitInstance: InstanceType = - isRemoteRepoOrOrgConfigured() ? getPATOctokit() : getDefaultOctokitClient() +export async function updateFile(sha: string, claFileContent, reactedCommitters: ReactedCommitterMap): Promise { - const pullRequestNo = context.issue.number - claFileContent?.signedContributors.push(...reactedCommitters.newSigned) - let contentString = JSON.stringify(claFileContent, null, 2) - let contentBinary = Buffer.from(contentString).toString('base64') - await octokitInstance.repos.createOrUpdateFileContents({ - owner: input.getRemoteOrgName() || context.repo.owner, - repo: input.getRemoteRepoName() || context.repo.repo, - path: input.getPathToSignatures(), - sha, - message: input.getSignedCommitMessage() - ? input - .getSignedCommitMessage() - .replace('$contributorName', context.actor) - : `@${context.actor} has signed the CLA from Pull Request #${pullRequestNo}`, - content: contentBinary, - branch: input.getBranch() - }) + const octokitInstance: InstanceType = + isRemoteRepoOrOrgConfigured() ? getPATOctokit() : getDefaultOctokitClient() + const pullRequestNo = context.issue.number + claFileContent?.signedContributors.push(...reactedCommitters.newSigned) + let contentString = JSON.stringify(claFileContent, null, 2) + let contentBinary = Buffer.from(contentString).toString("base64") + await octokitInstance.repos.createOrUpdateFileContents({ + owner: input.getRemoteOrgName() || context.repo.owner, + repo: input.getRemoteRepoName() || context.repo.repo, + path: input.getPathToSignatures(), + sha, + message: input.getSignedCommitMessage() + ? input + .getSignedCommitMessage() + .replace("$contributorName", context.actor) + .replace("$pullRequestNo", context.issue.number.toString()) + .replace("$owner", context.issue.owner) + .replace("$repo", context.issue.repo) + : `@${context.actor} has signed the CLA from Pull Request #${pullRequestNo}`, + content: contentBinary, + branch: input.getBranch() + }) } function isRemoteRepoOrOrgConfigured(): boolean {