From 85b82efdc6ca5654dad51425e6de33568db5e802 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 1 Apr 2024 02:38:58 -0700 Subject: [PATCH 01/10] chore: update used-by badge by github-actions[bot] (#228) Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index b7df919e..a74fb8b4 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=529&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=532&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From c07c25da31e23dc5aba5ec72284e5b7363c17f1a Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 8 Apr 2024 17:55:49 +0800 Subject: [PATCH 02/10] chore: update used-by badge by github-actions[bot] (#229) Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index a74fb8b4..d7ae9113 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=532&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=534&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From 60d879418396143b74b81438d3e9675f83319c35 Mon Sep 17 00:00:00 2001 From: Peter Shen Date: Mon, 8 Apr 2024 19:16:27 +0800 Subject: [PATCH 03/10] Swtich to `actions/stale` (#230) --- .github/stale.yml | 1 - .github/workflows/stale.yml | 10 ++++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) delete mode 100644 .github/stale.yml create mode 100644 .github/workflows/stale.yml diff --git a/.github/stale.yml b/.github/stale.yml deleted file mode 100644 index 0d0b1c99..00000000 --- a/.github/stale.yml +++ /dev/null @@ -1 +0,0 @@ -_extends: .github diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml new file mode 100644 index 00000000..952263f2 --- /dev/null +++ b/.github/workflows/stale.yml @@ -0,0 +1,10 @@ +name: 'Close stale issues' +on: + schedule: + - cron: '30 1 * * *' +permissions: + issues: write + +jobs: + stale: + uses: cpp-linter/.github/.github/workflows/stale.yml@main From bbc213852a439498b38fa21ea5c698e852abd3f5 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 15 Apr 2024 21:36:23 +0800 Subject: [PATCH 04/10] chore: update used-by badge by github-actions[bot] (#231) --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index d7ae9113..6455b605 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=534&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=539&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From a3ed7553c695735e6d1af356a0c0211a77d42c7b Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 22 Apr 2024 17:43:41 +0800 Subject: [PATCH 05/10] chore: update used-by badge by github-actions[bot] (#232) Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 6455b605..2425ca8e 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=539&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=545&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From 08fd7830fb2e8f842c5bdb89278409f4904fa70c Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 6 May 2024 17:13:34 -0400 Subject: [PATCH 06/10] chore: update used-by badge by github-actions[bot] (#235) --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 2425ca8e..ba361041 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=545&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=550&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From 8147325db2bef1f2ce985e84c1bf1bd02e6b2549 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Tue, 28 May 2024 15:03:13 +0800 Subject: [PATCH 07/10] chore: update used-by badge by github-actions[bot] (#241) Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index ba361041..5a68ea7f 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=550&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=565&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From 91cfe27ea9f72194d7a74c64bcd71f6613446cb1 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 3 Jun 2024 17:31:46 +0800 Subject: [PATCH 08/10] chore: update used-by badge by github-actions[bot] (#245) Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 5a68ea7f..61ccaef8 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ # C/C++ Linter Action | clang-format & clang-tidy ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/cpp-linter/cpp-linter-action) -[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=565&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) +[![Used by](https://img.shields.io/static/v1?label=Used%20by&message=573&color=informational&logo=slickpic)](https://github.com/cpp-linter/cpp-linter-action/network/dependents) [![GitHub marketplace](https://img.shields.io/badge/marketplace-C%2FC%2B%2B%20Linter-blue?logo=github)](https://github.com/marketplace/actions/c-c-linter) [![cpp-linter](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/cpp-linter.yml) [![MkDocs Deploy](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml/badge.svg)](https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml) From 09a8c476c173af944a6ae735d448dd67dc5c0f0e Mon Sep 17 00:00:00 2001 From: Brendan <2bndy5@gmail.com> Date: Wed, 5 Jun 2024 20:35:32 -0700 Subject: [PATCH 09/10] using homebrew to install clang tools on mac runner (#244) - manually symlink llvm binaries - continue despite homebrew error --- action.yml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/action.yml b/action.yml index 03dfff39..9590cba8 100644 --- a/action.yml +++ b/action.yml @@ -234,6 +234,15 @@ runs: fi fi + - name: Install MacOS clang dependencies + if: runner.os == 'macOS' + shell: bash + continue-on-error: true + run: | + brew install llvm@${{ inputs.version }} + ln -s "$(brew --prefix llvm@${{ inputs.version }})/bin/clang-format" "/usr/local/bin/clang-format-${{ inputs.version }}" + ln -s "$(brew --prefix llvm@${{ inputs.version }})/bin/clang-tidy" "/usr/local/bin/clang-tidy-${{ inputs.version }}" + - name: Setup python venv (Unix) if: runner.os == 'Linux' || runner.os == 'macOS' shell: bash From 820b1820ddd082ec68d266b7e6f47c65d78937bb Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri, 7 Jun 2024 18:02:09 -0700 Subject: [PATCH 10/10] Bump the pip group across 1 directory with 2 updates (#247) * Bump the pip group across 1 directory with 2 updates Bumps the pip group with 2 updates in the / directory: [clang-tools](https://github.com/cpp-linter/clang-tools-pip) and [cpp-linter](https://github.com/cpp-linter/cpp-linter). Updates `clang-tools` from 0.12.1 to 0.13.0 - [Release notes](https://github.com/cpp-linter/clang-tools-pip/releases) - [Commits](https://github.com/cpp-linter/clang-tools-pip/compare/v0.12.1...v0.13.0) Updates `cpp-linter` from 1.8.1 to 1.10.0 - [Release notes](https://github.com/cpp-linter/cpp-linter/releases) - [Commits](https://github.com/cpp-linter/cpp-linter/compare/v1.8.1...v1.10.0) --- updated-dependencies: - dependency-name: clang-tools dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip - dependency-name: cpp-linter dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip ... Signed-off-by: dependabot[bot] * add tool-specific ignore options resolves #233 * add input for passive reviews resolves #243 --------- Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Brendan <2bndy5@gmail.com> --- action.yml | 26 +++++++++++++++++++++++++- docs/action.yml | 7 +++++++ docs/permissions.md | 2 +- requirements.txt | 4 ++-- 4 files changed, 35 insertions(+), 4 deletions(-) diff --git a/action.yml b/action.yml index 9590cba8..a676f410 100644 --- a/action.yml +++ b/action.yml @@ -94,7 +94,20 @@ inputs: - Prefix a path with a bang (`!`) to make it explicitly _not_ ignored. The order of multiple paths does _not_ take precedence. The `!` prefix can be applied to a submodule's path (if desired) but not hidden directories. - - Glob patterns are not supported here. All asterisk characters (`*`) are literal. + - **As of v2.12**, glob patterns are supported here. + All asterisk characters (`*`) were previously literal. + required: false + default: '.github' + ignore-tidy: + description: |- + Use this option to allow clang-tidy to ignore certain paths/files. + See [`ignore`](#ignore) for more details on possible values. + required: false + default: '.github' + ignore-format: + description: |- + Use this option to allow clang-format to ignore certain paths/files. + See [`ignore`](#ignore) for more details on possible values. required: false default: '.github' thread-comments: @@ -191,6 +204,11 @@ inputs: > The [`no-lgtm`](#no-lgtm) option is applicable to Pull Request reviews. required: false default: false + passive-reviews: + description: | + Set this option to `true` to prevent Pull Request reviews from approving or requesting changes. + default: false + required: false jobs: description: | The number of jobs to run in parallel. @@ -272,11 +290,14 @@ runs: --no-lgtm=${{ inputs.no-lgtm }} \ --step-summary=${{ inputs.step-summary }} \ --ignore="${{ inputs.ignore }}" \ + --ignore-tidy="${{ inputs.ignore-tidy }}" \ + --ignore-format="${{ inputs.ignore-format }}" \ --database=${{ inputs.database }} \ --file-annotations=${{ inputs.file-annotations }} \ --extra-arg="${{ inputs.extra-args }}" \ --tidy-review="${{ inputs.tidy-review }}" \ --format-review="${{ inputs.format-review }}" \ + --passive-reviews="${{ inputs.passive-reviews }}" \ --jobs=${{ inputs.jobs }} - name: Setup python venv (Windows) @@ -308,11 +329,14 @@ runs: ' --no-lgtm=${{ inputs.no-lgtm }}' + ' --step-summary=${{ inputs.step-summary }}' + ' --ignore="${{ inputs.ignore }}"' + + ' --ignore-tidy="${{ inputs.ignore-tidy }}"' + + ' --ignore-format="${{ inputs.ignore-format }}"' + ' --database=${{ inputs.database }}' + ' --file-annotations=${{ inputs.file-annotations }}' + ' --extra-arg="${{ inputs.extra-args }}"' + ' --tidy-review="${{ inputs.tidy-review }}"' + ' --format-review="${{ inputs.format-review }}"' + + ' --passive-reviews="${{ inputs.passive-reviews }}"' + ' --jobs=${{ inputs.jobs }}' Invoke-Expression -Command $app diff --git a/docs/action.yml b/docs/action.yml index 40b65cd3..5e5c7982 100644 --- a/docs/action.yml +++ b/docs/action.yml @@ -20,6 +20,10 @@ inputs: required-permission: 'contents: read #file-changes' ignore: minimum-version: '1.3.0' + ignore-tidy: + minimum-version: '2.12.0' + ignore-format: + minimum-version: '2.12.0' thread-comments: minimum-version: '2.6.2' required-permission: 'contents: write #thread-comments' @@ -40,6 +44,9 @@ inputs: format-review: minimum-version: '2.9.0' required-permission: 'pull-requests: write #pull-request-reviews' + passive-reviews: + minimum-version: '2.12.0' + required-permission: 'pull-requests: write #pull-request-reviews' jobs: minimum-version: '2.11.0' outputs: diff --git a/docs/permissions.md b/docs/permissions.md index d10163f5..3495c392 100644 --- a/docs/permissions.md +++ b/docs/permissions.md @@ -72,7 +72,7 @@ The [`thread-comments`](inputs-outputs.md#thread-comments) feature requires the ## Pull Request Reviews -The [`tidy-review`](inputs-outputs.md#tidy-review) and [`format-review`](inputs-outputs.md#format-review) features require the following permissions: +The [`tidy-review`](inputs-outputs.md#tidy-review), [`format-review`](inputs-outputs.md#format-review), and [`passive-reviews`](inputs-outputs.md#passive-reviews) features require the following permissions: ```yaml permissions: diff --git a/requirements.txt b/requirements.txt index de7ed9f7..02917bcb 100644 --- a/requirements.txt +++ b/requirements.txt @@ -1,7 +1,7 @@ # Install clang-tools binaries (clang-format, clang-tidy) # For details please see: https://github.com/cpp-linter/clang-tools-pip -clang-tools==0.12.1 +clang-tools==0.13.0 # cpp-linter core Python executable package # For details please see: https://github.com/cpp-linter/cpp-linter -cpp-linter==1.8.1 +cpp-linter==1.10.0