From 6972ba1a5334a4b3c1abf0eac20fdfa317f34571 Mon Sep 17 00:00:00 2001 From: Nara Kasbergen Kwon <855115+xiehan@users.noreply.github.com> Date: Tue, 2 Jul 2024 13:17:37 +0200 Subject: [PATCH] chore: reenable required CLA checks (#296) --- lib/repository.ts | 4 +--- main.ts | 6 +++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/repository.ts b/lib/repository.ts index 85ed4f9..729fa01 100644 --- a/lib/repository.ts +++ b/lib/repository.ts @@ -42,9 +42,7 @@ export class RepositorySetup extends Construct { const { protectMain = false, - // @TODO: uncomment the below line if we want to require CLA checks again (and remove the line below it) - // protectMainChecks = ["build", "license/cla"], - protectMainChecks = ["build"], + protectMainChecks = ["build", "license/cla"], provider, repository, team, diff --git a/main.ts b/main.ts index 174cfb5..95ce9c0 100644 --- a/main.ts +++ b/main.ts @@ -167,6 +167,8 @@ class TerraformCdkProviderStack extends TerraformStack { { team: githubTeam, webhookUrl: slackWebhook.stringValue, + protectMain: true, + protectMainChecks: ["build", "package-js", "license/cla"], provider: githubProvider, }, ); @@ -271,9 +273,7 @@ class CustomConstructsStack extends TerraformStack { const secrets = new PublishingSecretSet(this, "secret-set"); constructRepos.forEach(({ name: repoName, languages, topics }) => { - // @TODO: uncomment the below line if we want to require CLA checks again (and remove the line below it) - // const protectMainChecks = ["build", "license/cla"].concat( - const protectMainChecks = ["build"].concat( + const protectMainChecks = ["build", "license/cla"].concat( languages.map((language) => { return `package-${ language === "typescript"