From 4c43031d5a719e285e543dd10a45cefeb5b12a9d Mon Sep 17 00:00:00 2001
From: Dylan Anthony <43723790+dbanty@users.noreply.github.com>
Date: Tue, 23 Nov 2021 10:29:00 -0700
Subject: [PATCH 1/2] Chore: Remove commit checker CI job

This job is inaccurate for PRs as we squash commits and is not automatically in sync with the eslint rules we currently use (nor will it sync with Angular when we switch).
---
 .github/workflows/commit_syntax.yml | 23 -----------------------
 1 file changed, 23 deletions(-)
 delete mode 100644 .github/workflows/commit_syntax.yml

diff --git a/.github/workflows/commit_syntax.yml b/.github/workflows/commit_syntax.yml
deleted file mode 100644
index f1fac5315..000000000
--- a/.github/workflows/commit_syntax.yml
+++ /dev/null
@@ -1,23 +0,0 @@
-name: 'Commit Message Check'
-on:
-  push:
-    branches: [ master, rc ]
-  pull_request:
-    branches: [ master, rc ]
-  workflow_dispatch:
-
-jobs:
-  check-commit-message:
-    name: Check Commit Message
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check Commit Type
-        uses: gsactions/commit-message-checker@v1
-        with:
-          pattern: '^((Breaking|Update|Fix|New|Upgrade|No code):|Release v.*)'
-          flags: ''
-          error: 'Your commit message should start with a valid semantic release verb as defined in package.json. You may also prepend "No code:" to indicate the commit does not contain changes to the released code.'
-          checkAllCommitMessages: true
-          excludeDescription: true
-          excludeTitle: true
-          accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true

From 534b990c67a60816f2fa347c4f66e1a9916a3f78 Mon Sep 17 00:00:00 2001
From: Dylan Anthony <43723790+dbanty@users.noreply.github.com>
Date: Tue, 23 Nov 2021 10:31:12 -0700
Subject: [PATCH 2/2] Chore: Remove commit syntax instructions from PR template

---
 .github/PULL_REQUEST_TEMPLATE.md | 2 --
 1 file changed, 2 deletions(-)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index b9423a022..45e8b89d0 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -7,7 +7,6 @@ Checklist
 ---------
 
 - [ ] The PR targets the `rc` branch (**not** `master`).
-- [ ] Commit messages are prefixed with one of the prefixes outlined in the [commit syntax checker][commit-syntax] (see `pattern` field).
 - [ ] The above description motivates these changes.
 - [ ] There is a unit test that covers these changes.
 - [ ] All new and existing tests pass locally and on the PR's checks.
@@ -22,7 +21,6 @@ Checklist
 [auto-close]: https://help.github.com/en/articles/closing-issues-using-keywords
 [black]: https://black.readthedocs.io/en/stable/index.html
 [changelog]: https://github.com/rigetti/pyquil/blob/master/CHANGELOG.md
-[commit-syntax]: https://github.com/rigetti/pyquil/blob/master/.github/workflows/commit_syntax.yml
 [contributing]: https://github.com/rigetti/pyquil/blob/master/CONTRIBUTING.md
 [docs]: https://pyquil.readthedocs.io
 [flake8]: http://flake8.pycqa.org