chore: reenable required CLA checks (#296)

cdktf · Jul 2, 2024 · 6972ba1 · 6972ba1
1 parent 60bef17
commit 6972ba1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git 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
@@ -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"