From 77b90f9e2be8007af89460c7ee3c42d11cd8a80f Mon Sep 17 00:00:00 2001 From: Jonathan Grahl Date: Fri, 16 Feb 2024 12:08:53 +0100 Subject: [PATCH 1/2] ci: test PRs --- .github/workflows/pr.yaml | 52 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 52 insertions(+) create mode 100644 .github/workflows/pr.yaml diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml new file mode 100644 index 0000000..8d8c2f7 --- /dev/null +++ b/.github/workflows/pr.yaml @@ -0,0 +1,52 @@ +name: "PR" + +on: + pull_request: + types: + - opened + - edited + - synchronize + +jobs: + pr-title-check: + name: Validate PR title + runs-on: ubuntu-latest + permissions: + pull-requests: read + steps: + - uses: amannn/action-semantic-pull-request@v5 + env: + GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} + - uses: marocchino/sticky-pull-request-comment@v2 + # When the previous steps fails, the workflow would stop. By adding this + # condition you can continue the execution with the populated error message. + if: always() && (steps.pr-title-lint.outputs.error_message != null) + id: pr-title-lint + with: + header: pr-title-lint-error + message: | + Hey there and thank you for opening this pull request! 👋🏼 + + We require pull request titles to follow the [Conventional Commits specification](https://www.conventionalcommits.org/en/v1.0.0/) and it looks like your proposed title needs to be adjusted. + + Details: + + ``` + ${{ steps.pr-title-lint.outputs.error_message }} + ``` + + # Delete a previous comment when the issue has been resolved + - if: ${{ steps.pr-title-lint.outputs.error_message == null }} + uses: marocchino/sticky-pull-request-comment@v2 + with: + header: pr-title-lint-error + delete: true + + build-and-test: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v3 + - name: Build + run: cargo build --verbose + - name: Run tests + run: cargo test --verbose From b0c1670ad369f15cc6dd34742c5fbf011d2cb510 Mon Sep 17 00:00:00 2001 From: Jonathan Grahl Date: Fri, 16 Feb 2024 12:19:13 +0100 Subject: [PATCH 2/2] fix: set name and remove verbose from build and test --- .github/workflows/pr.yaml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 8d8c2f7..dfce476 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -43,10 +43,11 @@ jobs: delete: true build-and-test: + name: Build and Test runs-on: ubuntu-latest steps: - uses: actions/checkout@v3 - name: Build - run: cargo build --verbose + run: cargo build - name: Run tests - run: cargo test --verbose + run: cargo test