Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: PR title/description change should not re-trigger the build-lint-test workflow #4100

Merged
merged 1 commit into from
Sep 14, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 32 additions & 0 deletions .github/workflows/check-pr-title.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
name: Test Pull Request Semantics

on:
pull_request:
types:
- opened
- edited
- synchronize

jobs:
check-pr-title:
runs-on: ubuntu-latest
permissions:
checks: write
pull-requests: read
contents: read
steps:
- uses: amannn/action-semantic-pull-request@v4
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
# https://www.notion.so/n8n/Release-Process-fce65faea3d5403a85210f7e7a60d0f8
types: |
feat
fix
perf
test
docs
refactor
build
ci
requireScope: false
33 changes: 1 addition & 32 deletions .github/workflows/ci-pull-requests.yml
Original file line number Diff line number Diff line change
@@ -1,38 +1,8 @@
name: Test Pull Requests

on:
pull_request:
types:
- opened
- edited
- synchronize
branches:
- master
on: [pull_request]

jobs:
check-pr-title:
runs-on: ubuntu-latest
permissions:
checks: write
pull-requests: read
contents: read
steps:
- uses: amannn/action-semantic-pull-request@v4
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
# https://www.notion.so/n8n/Release-Process-fce65faea3d5403a85210f7e7a60d0f8
types: |
feat
fix
perf
test
docs
refactor
build
ci
requireScope: false

build:
runs-on: ubuntu-latest

Expand All @@ -57,4 +27,3 @@ jobs:
npm run lint
env:
CI: true