fix: change lint title action (#1352)

* test bad pr title action change Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding current branch to run pr title action Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: changing settings for conventional pr title check Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: chaning pr lint flow Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correct permissions on folders Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding sudo to apt install Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding correct branch Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correcting typo Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correcting branch Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: add step to check Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * bad pr title test (#1356) * bad pr title test Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * spacing * fix: adding sudo to apt install Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * adding space Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * change readme Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * Update README.md * Update README.md --------- Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: fixing action condition Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correct conditional Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: add config conventional install Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding commitlint config Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: chaning branch to checkout Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * add validation just for title Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: grab the first commit msg Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding config Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: add pull request event Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correct action workflow Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: remove message Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding merge commit to check Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: geting latest commit Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: setup default branch Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: fixing syntax error Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: inverting commit order Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: commitlint config Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: fixing typo Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correcting typo Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: fixing typo Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: missing install Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding corrections Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: match pr title Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: add validation step Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: correct action Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: change pr title check steps Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: adding correct config Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> * fix: fix mdformat errors in readme * fix: target main and develop branches Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> --------- Signed-off-by: Alejandro Jose Leiva Palomo <[email protected]> Co-authored-by: manjiree-gadgil <[email protected]>
oscal-compass · May 12, 2023 · 5444206 · 5444206
1 parent d5da18d
commit 5444206
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 34 deletions.
diff --git a/.github/workflows/conventional-pr.yml b/.github/workflows/conventional-pr.yml
@@ -7,43 +7,20 @@ on:
       - edited
       - synchronize
     branches:
-      - 'develop'
       - 'main'
+      - 'develop'
 jobs:
   lint:
     if: ${{ github.head_ref != 'develop' }}
     runs-on: ubuntu-latest
     steps:
-      # Please look up the latest version from
-      # https://github.com/amannn/action-semantic-pull-request/releases
-      - uses: amannn/[email protected]
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          # Configure which types are allowed.
-          # Default: https://github.com/commitizen/conventional-commit-types
-          # Configure which scopes are allowed.
-          scopes: |
-            core
-            models
-            transformers
-            docs
-            cicd
-            author
-            cli
-          # Configure that a scope must always be provided.
-          requireScope: false
-          # Configure additional validation for the subject based on a regex.
-          # This example ensures the subject doesn't start with an uppercase character.
-          # For work-in-progress PRs you can typically use draft pull requests 
-          # from Github. However, private repositories on the free plan don't have 
-          # this option and therefore this action allows you to opt-in to using the 
-          # special "[WIP]" prefix to indicate this state. This will avoid the 
-          # validation of the PR title and the pull request checks remain pending.
-          # Note that a second check will be reported if this is enabled.
-          wip: true
-          # When using "Squash and merge" on a PR with only one commit, GitHub
-          # will suggest using that commit message instead of the PR title for the
-          # merge commit, and it's easy to commit this by mistake. Enable this option
-          # to also validate the commit
-          validateSingleCommit: true
+    - name: Checkout code
+      uses: actions/checkout@v2
+
+    - name: Install dependencies
+      run: npm install @commitlint/cli @commitlint/config-conventional
+
+    - name: Validate PR title
+      run: |
+        PR_TITLE=$(jq -r '.pull_request.title' "$GITHUB_EVENT_PATH")
+        echo "$PR_TITLE" | npx commitlint --config commitlint.config.js
diff --git a/commitlint.config.js b/commitlint.config.js
@@ -0,0 +1 @@
+module.exports = {extends: ['@commitlint/config-conventional']};
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		module.exports = {extends: ['@commitlint/config-conventional']};