feat: re-add branch protection toggling in release.yml (#144)

JoshuaKGoldberg · Dec 23, 2022 · b55395a · b55395a
1 parent a9249b8
commit b55395a
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 8 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -18,17 +18,61 @@ jobs:
       - env:
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
         run: npm config set //registry.npmjs.org/:_authToken $NPM_TOKEN
-      - name: Temporarily disable "include administrators" branch protection
-        uses: benjefferies/branch-protection-bot@1.0.7
+      - name: Delete branch protection on main
+        uses: actions/github-script@v6.3.3
         with:
-          access_token: ${{ secrets.ACCESS_TOKEN }}
-          branch: ${{ github.event.repository.default_branch }}
+          github-token: ${{ secrets.ACCESS_TOKEN }}
+          script: |
+            try {
+              await github.request(
+                `DELETE /repos/JoshuaKGoldberg/template-typescript-node-package/branches/main/protection`,
+              );
+            } catch (error) {
+              if (!error.message?.includes?.("Branch not protected")) {
+                throw error;
+              }
+            }
       - env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: pnpm release-it
       - if: always()
-        name: Enable "include administrators" branch protection
-        uses: benjefferies/branch-protection-bot@1.0.7
+        name: Recreate branch protection on main
+        uses: actions/github-script@v6.3.3
         with:
-          access_token: ${{ secrets.ACCESS_TOKEN }}
-          branch: ${{ github.event.repository.default_branch }}
+          github-token: ${{ secrets.ACCESS_TOKEN }}
+          # Note: keep this inline script in sync with script/setup.js!
+          # Todo: it would be nice to not have two sources of truth...
+          script: |
+            github.request(
+              `PUT /repos/JoshuaKGoldberg/template-typescript-node-package/branches/main/protection`,
+              {
+                allow_deletions: false,
+                allow_force_pushes: true,
+                allow_fork_pushes: false,
+                allow_fork_syncing: true,
+                block_creations: false,
+                branch: "main",
+                enforce_admins: false,
+                owner: "JoshuaKGoldberg",
+                repo: "template-typescript-node-package",
+                required_conversation_resolution: true,
+                required_linear_history: false,
+                required_pull_request_reviews: null,
+                required_status_checks: {
+                  checks: [
+                    { context: "build" },
+                    { context: "compliance" },
+                    { context: "lint" },
+                    { context: "markdown" },
+                    { context: "package" },
+                    { context: "packages" },
+                    { context: "prettier" },
+                    { context: "prune" },
+                    { context: "spelling" },
+                    { context: "test" },
+                  ],
+                  strict: false,
+                },
+                restrictions: null,
+              }
+            );
diff --git a/script/setup.js b/script/setup.js
@@ -197,6 +197,8 @@ try {
 	console.log();
 	console.log(chalk.gray`Hydrating branch protection settings...`);
 
+	// Note: keep this inline script in sync with .github/workflows/release.yml!
+	// Todo: it would be nice to not have two sources of truth...
 	await octokit.request(
 		`PUT /repos/${owner}/${repository}/branches/main/protection`,
 		{