From a8f562b04792b39c6c14afea09145c9ec7151fae Mon Sep 17 00:00:00 2001 From: Peter Newman Date: Sat, 6 Jun 2020 02:12:30 +0100 Subject: [PATCH 1/4] Codespell our action --- .github/workflows/testing.yml | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml index d1297d1..cf8438d 100644 --- a/.github/workflows/testing.yml +++ b/.github/workflows/testing.yml @@ -17,7 +17,7 @@ jobs: steps: - name: Checkout - uses: actions/checkout@v1 + uses: actions/checkout@v2 - name: Set up Python uses: actions/setup-python@v2 @@ -41,12 +41,22 @@ jobs: name: Test run action runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v2 - uses: ./ with: path: test/testdata only_warn: 1 + codespell: + name: Check for spelling errors + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v2 + - uses: ./ + with: + check_filenames: true + check_hidden: true + diagnose_bats: name: Diagnose bats runs-on: ubuntu-latest From 3d7936c5bd39df83b45cf30deb9dcc72b9e8ddb2 Mon Sep 17 00:00:00 2001 From: Peter Newman Date: Sat, 6 Jun 2020 02:18:46 +0100 Subject: [PATCH 2/4] Skip some test folders --- .github/workflows/testing.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml index cf8438d..c811e55 100644 --- a/.github/workflows/testing.yml +++ b/.github/workflows/testing.yml @@ -56,6 +56,8 @@ jobs: with: check_filenames: true check_hidden: true + # When using this Action in other repos, the --skip option below can be removed + skip: ./.git/,./codespell-problem-matcher/test/,./test/ diagnose_bats: name: Diagnose bats From 0b02c63f023fc13b13be495ea150b5d10684f462 Mon Sep 17 00:00:00 2001 From: Peter Newman Date: Sat, 6 Jun 2020 02:19:49 +0100 Subject: [PATCH 3/4] Try fixing our skip config --- .github/workflows/testing.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml index c811e55..422339a 100644 --- a/.github/workflows/testing.yml +++ b/.github/workflows/testing.yml @@ -57,7 +57,7 @@ jobs: check_filenames: true check_hidden: true # When using this Action in other repos, the --skip option below can be removed - skip: ./.git/,./codespell-problem-matcher/test/,./test/ + skip: ./.git,./codespell-problem-matcher/test,./test diagnose_bats: name: Diagnose bats From 08fe393e92a4210eef4e9badbdf3eadf2e58fb02 Mon Sep 17 00:00:00 2001 From: Peter Newman Date: Sat, 6 Jun 2020 02:24:40 +0100 Subject: [PATCH 4/4] Fix the config typos codespell found --- action.yml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/action.yml b/action.yml index e65adf0..16315cd 100644 --- a/action.yml +++ b/action.yml @@ -4,27 +4,27 @@ description: 'Codespell with annotations for Pull Request' inputs: check_filenames: description: 'If set, check file names as well' - requred: false + required: false default: '' check_hidden: description: 'If set, check hidden files (those starting with ".") as well' - requred: false + required: false default: '' exclude_file: description: 'FILE with lines that should not be changed' - requred: false + required: false default: '' skip: description: 'Comma-separated list of files to skip (it accepts globs as well)' - requred: false + required: false default: '' path: description: 'Path to run codespell in' - requred: false + required: false default: '' only_warn: description: 'If set, only warn, never error' - requred: false + required: false default: '' runs: using: 'docker'