Skip to content

Allow action to be used in multiple jobs#43

Open
colluca wants to merge 1 commit intochipsalliance:mainfrom
colluca:allow-multiple-jobs
Open

Allow action to be used in multiple jobs#43
colluca wants to merge 1 commit intochipsalliance:mainfrom
colluca:allow-multiple-jobs

Conversation

@colluca
Copy link
Copy Markdown

@colluca colluca commented Jan 30, 2026

Previously using the action in multiple jobs as in:

  verible-lint:
    name: Lint Verilog sources
    runs-on: ubuntu-latest
    if: >
      github.event_name != 'pull_request' ||
      github.event.pull_request.head.repo.full_name != github.repository
    steps:
      - uses: actions/checkout@v4
      - uses: colluca/verible-linter-action@main
        with:
          paths: |
            ./hw
          github_token: ${{ secrets.GITHUB_TOKEN }}
          fail_on_error: true
          reviewdog_reporter: github-check
          extra_args: "--waiver_files util/lint/waiver.verible"
          verible_version: "v0.0-3318-g8d254167"

  strict-verible-lint:
    name: Lint Verilog sources (strict)
    runs-on: ubuntu-latest
    if: >
      github.event_name != 'pull_request' ||
      github.event.pull_request.head.repo.full_name != github.repository
    steps:
      - uses: actions/checkout@v4
      - uses: colluca/verible-linter-action@main
        with:
          config_file: './util/lint/rules.verible'
          paths: |
            ./hw/schnizo
          github_token: ${{ secrets.GITHUB_TOKEN }}
          fail_on_error: true
          reviewdog_reporter: github-check
          extra_args: "--waiver_files util/lint/waiver.verible"
          verible_version: "v0.0-4051-g9fdb4057"

Would raise a Failed to CreateArtifact error, due to a conflict in the artifact name:

Conflict: an artifact with this name already exists on the workflow run

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Jan 30, 2026

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: colluca / name: Luca Colagrande (655a8f7)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant