From 7030d6c1babbd58fcd083e7e0aa375bb01c41f35 Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Mon, 27 Nov 2023 09:56:04 -0500 Subject: [PATCH] build: Updating workflow `commitlint.yml`. The .github/workflows/commitlint.yml workflow is missing or needs an update to stay in sync with the current standard for this workflow as defined in the `.github` repo of the `openedx` GitHub org. --- .github/workflows/commitlint.yml | 35 +------------------------------- 1 file changed, 1 insertion(+), 34 deletions(-) diff --git a/.github/workflows/commitlint.yml b/.github/workflows/commitlint.yml index 23163aa64..fec11d6c2 100644 --- a/.github/workflows/commitlint.yml +++ b/.github/workflows/commitlint.yml @@ -1,43 +1,10 @@ # Run commitlint on the commit messages in a pull request. -# -# This is the same worflow of -# https://github.com/openedx/.github/blob/master/.github/workflows/commitlint.yml -# -# We are using typescript with react-query on this MFE, and it is necessary -# to add one more step to remove tsconfig.json due to an issue in commitlint. -# https://github.com/conventional-changelog/commitlint/issues/3256 -# -# We can return use the open edx workflow when the issue is fixed. name: Lint Commit Messages on: - pull_request -defaults: - run: - shell: bash - jobs: commitlint: - runs-on: ubuntu-20.04 - steps: - - name: Check out the repo - uses: actions/checkout@v4 - with: - # Fetch 100 commits. Should be enough? - fetch-depth: 100 - - - name: Download a local configuration file if needed - run: | - if [[ ! -f commitlint.config.js ]]; then - echo "Downloading the default commitlint config from edx_lint" - wget --no-verbose -O commitlint.config.js https://raw.githubusercontent.com/openedx/edx-lint/HEAD/edx_lint/files/commitlint.config.js - fi - - - name: remove tsconfig.json # see issue https://github.com/conventional-changelog/commitlint/issues/3256 - run: | - rm -f tsconfig.json - - - name: Run commitlint - uses: wagoid/commitlint-github-action@v5 + uses: openedx/.github/.github/workflows/commitlint.yml@master