PR(FIX): Skip change-detector tests with any uncommitted transaction #938
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| # Copyright 2023 Democratized Data Foundation | |
| # | |
| # Use of this software is governed by the Business Source License | |
| # included in the file licenses/BSL.txt. | |
| # | |
| # As of the Change Date specified in that file, in accordance with | |
| # the Business Source License, use of this software will be governed | |
| # by the Apache License, Version 2.0, included in the file | |
| # licenses/APL.txt. | |
| name: Lint Workflow | |
| on: | |
| pull_request: | |
| branches: | |
| - master | |
| - develop | |
| push: | |
| permissions: | |
| # Required for the `only-new-issues` option. | |
| pull-requests: read | |
| # Required for analysis. | |
| contents: read | |
| # Required to annotate code in the PR. | |
| checks: write | |
| jobs: | |
| lint-go: | |
| name: Lint GoLang job | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Checkout code into the directory | |
| uses: actions/checkout@v6 | |
| # Setting up Go explicitly is required for v3.0.0+ of golangci/golangci-lint-action. | |
| - name: Setup Go environment explicitly | |
| uses: actions/setup-go@v6 | |
| with: | |
| go-version-file: 'go.mod' | |
| check-latest: true | |
| cache: false | |
| - name: Run golangci-lint linter | |
| uses: golangci/golangci-lint-action@v9 | |
| with: | |
| # Required: the version of golangci-lint is required. | |
| # Note: The version should not pick the patch version as the latest patch | |
| # version is what will always be used. | |
| version: v2.3 | |
| # The mode to install golangci-lint. Default is `binary`, but we had problems | |
| # with `binary` in the past where the binary was built with a higher version | |
| # of go-lang that we didn't support, causing errors. | |
| install-mode: goinstall | |
| # Optional: working directory, useful for monorepos or if we wanted to run this | |
| # on a non-root directory. | |
| # working-directory: ./ | |
| # Optional: golangci-lint command line arguments. | |
| # Note: we can set `--issues-exit-code=0` if we want a success-code always, | |
| # indicating that the linter ran successfully (weather or not linter errors | |
| # exist or not doesn't matter). But the good thing is that the annotations | |
| # will still show up. This can be useful if we don't want the pipeline to | |
| # stop just because we had some linter errors. Adding a = between the flag | |
| # name and its value is important because the action parses the arguments on | |
| # spaces. | |
| args: --issues-exit-code=1 --config=tools/configs/golangci.yaml | |
| # Optional: we can set the below to `true` if we only want to see newly | |
| # introduced linter errors, however I found that in practive that option is a | |
| # bit gimmicky, as it passes the linter check despite having new linter errors | |
| # in some cases. So we opt in for all annotations of linter errors to show up, | |
| # this is actually nicer because we suppress our linter errors manually | |
| # anyways so there shouldn't be any linter errors anyways. The enforces us to | |
| # always have a clean lint state. | |
| only-new-issues: false | |
| lint-yaml: | |
| name: Lint YAML job | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Checkout code into the directory | |
| uses: actions/checkout@v6 | |
| - name: Run yamllint linter | |
| uses: ibiqlik/action-yamllint@v3 | |
| with: | |
| config_file: tools/configs/yamllint.yaml | |
| file_or_dir: . |