Merge pull request #1760 from Flow-Launcher/StopSpellCheckOnPush

Stop spell check on push
This commit is contained in:
Jeremy Wu 2023-01-05 13:47:31 +11:00 committed by GitHub
commit 079dfd7faa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -34,12 +34,12 @@ name: Check Spelling
# For background, see: https://github.com/check-spelling/check-spelling/wiki/Feature:-Update-with-deploy-key # For background, see: https://github.com/check-spelling/check-spelling/wiki/Feature:-Update-with-deploy-key
on: on:
push: # push:
branches: # branches:
- '**' # - '**'
- '!l10n_dev' # - '!l10n_dev'
tags-ignore: # tags-ignore:
- "**" # - "**"
pull_request_target: pull_request_target:
branches: branches:
- '**' - '**'
@ -65,7 +65,7 @@ jobs:
outputs: outputs:
followup: ${{ steps.spelling.outputs.followup }} followup: ${{ steps.spelling.outputs.followup }}
runs-on: ubuntu-latest runs-on: ubuntu-latest
if: (contains(github.event_name, 'pull_request') && github.head_ref != 'l10n_dev') || github.event_name == 'push' if: (contains(github.event_name, 'pull_request') && github.head_ref != 'l10n_dev')
concurrency: concurrency:
group: spelling-${{ github.event.pull_request.number || github.ref }} group: spelling-${{ github.event.pull_request.number || github.ref }}
# note: If you use only_check_changed_files, you do not want cancel-in-progress # note: If you use only_check_changed_files, you do not want cancel-in-progress
@ -103,21 +103,21 @@ jobs:
comment-push: # comment-push:
name: Report (Push) # name: Report (Push)
# If your workflow isn't running on push, you can remove this job # # If your workflow isn't running on push, you can remove this job
runs-on: ubuntu-latest # runs-on: ubuntu-latest
needs: spelling # needs: spelling
permissions: # permissions:
contents: write # contents: write
if: (success() || failure()) && needs.spelling.outputs.followup && github.event_name == 'push' # if: (success() || failure()) && needs.spelling.outputs.followup && github.event_name == 'push'
steps: # steps:
- name: comment # - name: comment
uses: check-spelling/check-spelling@main # uses: check-spelling/check-spelling@main
with: # with:
checkout: true # checkout: true
spell_check_this: check-spelling/spell-check-this@main # spell_check_this: check-spelling/spell-check-this@main
task: ${{ needs.spelling.outputs.followup }} # task: ${{ needs.spelling.outputs.followup }}
comment-pr: comment-pr:
name: Report (PR) name: Report (PR)