From 697c0a54b48410ea6b2020dd3c9a09f55e4ce95d Mon Sep 17 00:00:00 2001 From: Fernandez Ludovic Date: Wed, 12 Jun 2024 05:26:17 +0200 Subject: [PATCH] chore: add test installation on the CI --- .github/workflows/post-release.yml | 10 ++++++++++ .github/workflows/pr.yml | 11 +++++++++++ .golangci.reference.yml | 7 ++++--- 3 files changed, 25 insertions(+), 3 deletions(-) diff --git a/.github/workflows/post-release.yml b/.github/workflows/post-release.yml index 90edab56c2f8..a36ee0a6408c 100644 --- a/.github/workflows/post-release.yml +++ b/.github/workflows/post-release.yml @@ -103,3 +103,13 @@ jobs: branch-suffix: timestamp title: "docs: update documentation assets" delete-branch: true + + check-install-script: + name: Installation script (remote) + strategy: + matrix: + os: [ubuntu-latest, macos-latest, windows-latest] + runs-on: ${{ matrix.os }} + + steps: + - run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b "./install-golangci-lint" diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml index b0c6e9173736..aade5b4bb190 100644 --- a/.github/workflows/pr.yml +++ b/.github/workflows/pr.yml @@ -128,6 +128,17 @@ jobs: # needed for github-action-config.json generation GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} + check-local-install-script: + name: Installation script (local) + strategy: + matrix: + os: [ubuntu-latest, macos-latest, windows-latest] + runs-on: ${{ matrix.os }} + steps: + - uses: actions/checkout@v4 + - name: Check installation script + run: cat ./install.sh | sh -s -- -d -b "./install-golangci-lint" + # Note: the command `run` is tested by the previous steps (`make test`). commands: needs: golangci-lint diff --git a/.golangci.reference.yml b/.golangci.reference.yml index c87aec032978..21d3be9abce4 100644 --- a/.golangci.reference.yml +++ b/.golangci.reference.yml @@ -2944,13 +2944,14 @@ issues: # Default: "" new-from-patch: path/to/patch/file + # Show issues in any part of update files (requires new-from-rev or new-from-patch). + # Default: false + whole-files: true + # Fix found issues (if it's supported by the linter). # Default: false fix: true - # Show issues in any part of update files (requires new-from-rev or new-from-patch). - # Default: false - whole-files: true severity: # Set the default severity for issues.