Skip to content

Commit

Permalink
Removes steps to disable and enable branch protection, since the NR O…
Browse files Browse the repository at this point in the history
…penSource bot token can do direct pushes to master
  • Loading branch information
jsubirat committed Sep 26, 2024
1 parent b2f1c86 commit 5555ed8
Showing 1 changed file with 1 addition and 81 deletions.
82 changes: 1 addition & 81 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,52 +75,12 @@ jobs:
echo "No change in package.json, not regenerating third-party notices"
fi
- name: Temporarily disable "required_pull_request_reviews" branch protection
id: disable-branch-protection
if: always()
uses: actions/github-script@v1
with:
github-token: ${{ secrets.OPENSOURCE_BOT_TOKEN }}
previews: luke-cage-preview
script: |
const result = await github.repos.updateBranchProtection({
owner: context.repo.owner,
repo: context.repo.repo,
branch: 'master',
required_status_checks: null,
restrictions: null,
enforce_admins: null,
required_pull_request_reviews: null
})
console.log("Result:", result)
- name: Push Commit
if: steps.generate-notices.outputs.commit == 'true'
uses: ad-m/[email protected]
with:
github_token: ${{ secrets.OPENSOURCE_BOT_TOKEN }}

- name: Re-enable "required_pull_request_reviews" branch protection
id: enable-branch-protection
if: always()
uses: actions/github-script@v1
with:
github-token: ${{ secrets.OPENSOURCE_BOT_TOKEN }}
previews: luke-cage-preview
script: |
const result = await github.repos.updateBranchProtection({
owner: context.repo.owner,
repo: context.repo.repo,
branch: 'master',
required_status_checks: null,
restrictions: null,
enforce_admins: null,
required_pull_request_reviews: {
required_approving_review_count: 1
}
})
console.log("Result:", result)
job-generate-release:
runs-on: ubuntu-latest
needs: [job-generate-third-party-notices]
Expand Down Expand Up @@ -151,47 +111,7 @@ jobs:
- name: Install dependencies
run: npm ci

- name: Temporarily disable "required_pull_request_reviews" branch protection
id: disable-branch-protection
if: always()
uses: actions/github-script@v1
with:
github-token: ${{ secrets.OPENSOURCE_BOT_TOKEN }}
previews: luke-cage-preview
script: |
const result = await github.repos.updateBranchProtection({
owner: context.repo.owner,
repo: context.repo.repo,
branch: 'master',
required_status_checks: null,
restrictions: null,
enforce_admins: null,
required_pull_request_reviews: null
})
console.log("Result:", result)
- name: Run semantic-release
env:
GITHUB_TOKEN: ${{ secrets.OPENSOURCE_BOT_TOKEN }}
run: npx semantic-release

- name: Re-enable "required_pull_request_reviews" branch protection
id: enable-branch-protection
if: always()
uses: actions/github-script@v1
with:
github-token: ${{ secrets.OPENSOURCE_BOT_TOKEN }}
previews: luke-cage-preview
script: |
const result = await github.repos.updateBranchProtection({
owner: context.repo.owner,
repo: context.repo.repo,
branch: 'master',
required_status_checks: null,
restrictions: null,
enforce_admins: null,
required_pull_request_reviews: {
required_approving_review_count: 1
}
})
console.log("Result:", result)
run: npx semantic-release

0 comments on commit 5555ed8

Please sign in to comment.