Do not fail diff informed analyses when analyze is run twice in the same job #3160
Workflow file for this run
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
| # Warning: This file is generated automatically, and should not be modified. | |
| # Instead, please modify the template in the pr-checks directory and run: | |
| # (cd pr-checks; pip install [email protected] && python3 sync.py) | |
| # to regenerate this file. | |
| name: 'PR Check - Go: diagnostic when Go is changed after init step' | |
| env: | |
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |
| GO111MODULE: auto | |
| on: | |
| push: | |
| branches: | |
| - main | |
| - releases/v* | |
| pull_request: | |
| types: | |
| - opened | |
| - synchronize | |
| - reopened | |
| - ready_for_review | |
| schedule: | |
| - cron: '0 5 * * *' | |
| workflow_dispatch: {} | |
| jobs: | |
| go-indirect-tracing-workaround-diagnostic: | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| include: | |
| - os: ubuntu-latest | |
| version: default | |
| name: 'Go: diagnostic when Go is changed after init step' | |
| permissions: | |
| contents: read | |
| security-events: read | |
| timeout-minutes: 45 | |
| runs-on: ${{ matrix.os }} | |
| steps: | |
| - name: Check out repository | |
| uses: actions/checkout@v4 | |
| - name: Prepare test | |
| id: prepare-test | |
| uses: ./.github/actions/prepare-test | |
| with: | |
| version: ${{ matrix.version }} | |
| use-all-platform-bundle: 'false' | |
| setup-kotlin: 'true' | |
| - uses: actions/setup-go@v5 | |
| with: | |
| # We need a Go version that ships with statically linked binaries on Linux | |
| go-version: '>=1.21.0' | |
| - uses: ./../action/init | |
| with: | |
| languages: go | |
| tools: ${{ steps.prepare-test.outputs.tools-url }} | |
| # Deliberately change Go after the `init` step | |
| - uses: actions/setup-go@v5 | |
| with: | |
| go-version: '1.20' | |
| - name: Build code | |
| shell: bash | |
| run: go build main.go | |
| - uses: ./../action/analyze | |
| with: | |
| output: ${{ runner.temp }}/results | |
| upload-database: false | |
| - name: Check diagnostic appears in SARIF | |
| uses: actions/github-script@v7 | |
| env: | |
| SARIF_PATH: ${{ runner.temp }}/results/go.sarif | |
| with: | |
| script: | | |
| const fs = require('fs'); | |
| const sarif = JSON.parse(fs.readFileSync(process.env['SARIF_PATH'], 'utf8')); | |
| const run = sarif.runs[0]; | |
| const toolExecutionNotifications = run.invocations[0].toolExecutionNotifications; | |
| const statusPageNotifications = toolExecutionNotifications.filter(n => | |
| n.descriptor.id === 'go/workflow/go-installed-after-codeql-init' && n.properties?.visibility?.statusPage | |
| ); | |
| if (statusPageNotifications.length !== 1) { | |
| core.setFailed( | |
| 'Expected exactly one status page reporting descriptor for this diagnostic in the ' + | |
| `'runs[].invocations[].toolExecutionNotifications[]' SARIF property, but found ` + | |
| `${statusPageNotifications.length}. All notification reporting descriptors: ` + | |
| `${JSON.stringify(toolExecutionNotifications)}.` | |
| ); | |
| } | |
| env: | |
| CODEQL_ACTION_TEST_MODE: true |