Switch to dev branch of clang-tidy-review (#3937)

This supports the split workflow logic which makes it work on fork PRs
This commit is contained in:
pajlada 2022-08-28 13:32:07 +02:00 committed by GitHub
parent dcd42cb28b
commit 2fd962261b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 49 additions and 1 deletions

View file

@ -149,11 +149,20 @@ jobs:
- name: clang-tidy review - name: clang-tidy review
if: (startsWith(matrix.os, 'ubuntu') && matrix.pch == false && matrix.qt-version == '5.15.2' && github.event_name == 'pull_request') if: (startsWith(matrix.os, 'ubuntu') && matrix.pch == false && matrix.qt-version == '5.15.2' && github.event_name == 'pull_request')
uses: ZedThree/clang-tidy-review@v0.9.0 uses: pajlada/clang-tidy-review@feat/split-up-review-and-post-workflows
id: review id: review
with: with:
build_dir: build build_dir: build
config_file: '.clang-tidy' config_file: '.clang-tidy'
split_workflow: true
- uses: actions/upload-artifact@v3
if: (startsWith(matrix.os, 'ubuntu') && matrix.pch == false && matrix.qt-version == '5.15.2' && github.event_name == 'pull_request')
with:
name: clang-tidy-review
path: |
clang-tidy-review-output.json
clang-tidy-review-metadata.json
- name: Package - AppImage (Ubuntu) - name: Package - AppImage (Ubuntu)
if: startsWith(matrix.os, 'ubuntu') if: startsWith(matrix.os, 'ubuntu')

View file

@ -0,0 +1,39 @@
---
name: Post clang-tidy review comments
on:
workflow_run:
workflows: ["Build"]
types:
- completed
jobs:
build:
runs-on: ubuntu-latest
steps:
- name: 'Download artifact'
uses: actions/github-script@v6
with:
script: |
const artifacts = await github.rest.actions.listWorkflowRunArtifacts({
owner: context.repo.owner,
repo: context.repo.repo,
run_id: ${{github.event.workflow_run.id }},
});
const matchArtifact = artifacts.data.artifacts.filter((artifact) => {
return artifact.name == "clang-tidy-review"
})[0];
const download = await github.rest.actions.downloadArtifact({
owner: context.repo.owner,
repo: context.repo.repo,
artifact_id: matchArtifact.id,
archive_format: 'zip',
});
const fs = require('fs');
fs.writeFileSync('${{github.workspace}}/clang-tidy-review.zip', Buffer.from(download.data));
- name: 'Unzip artifact'
run: unzip clang-tidy-review.zip
- uses: pajlada/clang-tidy-review/post@feat/split-up-review-and-post-workflows
id: review