From eca9c1e539dc715270105ead81f6f2b59eeec5ee Mon Sep 17 00:00:00 2001 From: "K.B.Dharun Krishna" Date: Tue, 12 Sep 2023 21:28:12 +0530 Subject: [PATCH] CI/codespell: remove changed-files action, use point release of action (#10707) * CI/codespell: remove changed-files action Superseeds #10704 This change would make the CI run only on English pages by directly passing the path to Codespell to prevent false detections like https://github.com/tldr-pages/tldr/pull/10702/files for other languages, There still would be annotations made for other English pages not in PR similar to now, but it won't prevent us from merging the PR as we have the only_warn parameter set to 1. * CI/codespell: use point release instead of master Using the development branch isn't ideal for production as it is prone to breakage with a faulty bug or PR. So in this commit, I am updating the codespell action to use the point release instead for more stability and versatility. --- .github/workflows/codespell.yml | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/.github/workflows/codespell.yml b/.github/workflows/codespell.yml index 8ce2d0c4d..a75562cef 100644 --- a/.github/workflows/codespell.yml +++ b/.github/workflows/codespell.yml @@ -9,19 +9,10 @@ jobs: steps: - uses: actions/checkout@v4 - - name: Get changed files - id: changed-files - uses: tj-actions/changed-files@v39.0.0 - with: - # Ignore all other languages except English - files_ignore: | - pages.*/*/* - package-lock.json - - - uses: codespell-project/actions-codespell@master + - uses: codespell-project/actions-codespell@v2 with: ignore_words_file: .github/codespell-ignore # Exit with 0 regardless of typos. only_warn: 1 - # Only check files in the PR - path: ${{ steps.changed-files.outputs.all_changed_files }} + # Only check English files from the pages directory + path: pages/*