Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit 32e6868

Browse files
authored
Respect commit SHA if provided (cloudposse#19)
1 parent 17f0566 commit 32e6868

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

dist/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62260,7 +62260,7 @@ class GetTerraformPlanUseCase {
6226062260
const { commitSHA, component, isMergeCommit, stack, planPath, pr, repoName, repoOwner, } = req;
6226162261
let plan;
6226262262
let metadata;
62263-
if (isMergeCommit) {
62263+
if (commitSHA && commitSHA != '' && isMergeCommit) {
6226462264
if (!pr) {
6226562265
return (0, infrastructure_1.left)(new infrastructure_1.AppError.UnexpectedError("PR is required for merge commits"));
6226662266
}

dist/index.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/useCases/getPlan/useCase.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ export class GetTerraformPlanUseCase
7070
let plan: Uint8Array;
7171
let metadata: TerraformPlan;
7272

73-
if (isMergeCommit) {
73+
if (commitSHA && commitSHA != '' && isMergeCommit) {
7474
if (!pr) {
7575
return left(
7676
new AppError.UnexpectedError("PR is required for merge commits")

0 commit comments

Comments
 (0)