Skip to content

Commit

Permalink
Update golangci-lint action to go through all dirs if no working dire…
Browse files Browse the repository at this point in the history
…ctory set
  • Loading branch information
chudilka1 committed Nov 26, 2024
1 parent 959ab5a commit 04be7f7
Showing 1 changed file with 29 additions and 6 deletions.
35 changes: 29 additions & 6 deletions .github/workflows/ci-core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,24 +91,47 @@ jobs:
- '!nix-darwin-shell-hook.sh'
- '!LICENSE'
- '!.github/**'
# Golangci-lint needs path to each module to run in monorepo,
# so we resolve the modules first
resolve-modules:
name: Resolve Modules for GolangCI Lint
runs-on: ubuntu-latest
outputs:
matrix: ${{ steps.set-modules.outputs.matrix }}
steps:
- name: Checkout
uses: actions/checkout@v4.2.1
- name: Set Modules for GolangCI Lint
id: set-modules
run: |
PATHS_TO_GO_MOD=$(find . -type f -name go.mod | awk -F'/go.mod' '{if ($1 == ".") print "./"; else print $1}')
PATHS_TO_GO_MOD_JSON=$(echo "$PATHS_TO_GO_MOD" | jq -R -s -c 'split("\n")[:-1]')
echo "matrix=${PATHS_TO_GO_MOD_JSON}" >> $GITHUB_OUTPUT
golangci:
name: lint
# We don't directly merge dependabot PRs, so let's not waste the resources
if: ${{ (github.event_name == 'pull_request' || github.event_name == 'schedule') && github.actor != 'dependabot[bot]' }}
name: lint
needs: [filter, run-frequency, resolve-modules]
permissions:
# For golangci-lint-actions to annotate code in the PR.
checks: write
contents: read
# For golangci-lint-action's `only-new-issues` option.
pull-requests: read
runs-on: ubuntu-24.04-8cores-32GB-ARM
needs: [filter, run-frequency]
strategy:
matrix:
modules: ${{fromJson(needs.resolve-modules.outputs.matrix)}}
steps:
- uses: actions/checkout@v4.2.1
- name: Golang Lint
uses: ./.github/actions/golangci-lint
- name: Checkout
uses: actions/checkout@v4.2.1
- name: Golang Lint (${{ matrix.modules }})
if: ${{ needs.filter.outputs.should-run-golangci == 'true' }}
uses: ./actions/golangci-lint
with:
go-directory: ${{ matrix.modules }}
- name: Notify Slack
if: ${{ failure() && needs.run-frequency.outputs.one-per-day-frequency == 'true' }}
uses: slackapi/slack-github-action@6c661ce58804a1a20f6dc5fbee7f0381b469e001 # v1.25.0
Expand Down

0 comments on commit 04be7f7

Please sign in to comment.