Skip to content

ci: pin cpplint reviewdog action - #31653

Open
zcxGGmu wants to merge 1 commit into
microsoft:mainfrom
zcxGGmu:pin-cpplint-action
Open

ci: pin cpplint reviewdog action#31653
zcxGGmu wants to merge 1 commit into
microsoft:mainfrom
zcxGGmu:pin-cpplint-action

Conversation

@zcxGGmu

@zcxGGmu zcxGGmu commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • Pin reviewdog/action-cpplint to a full commit SHA instead of the mutable master branch.
  • Keep the existing lint-cpp step configuration unchanged.

Fixes #29710

Test Plan

  • python check that reviewdog/action-cpplint@master is removed and the action is pinned to a 40-character SHA
  • python YAML parse check that the cpplint step and github_token configuration are unchanged apart from the action ref
  • git diff --check
  • diff secret/conflict scan (github_token line is unchanged and expected)

Signed-off-by: zq <zhouquan1511@163.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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.

CI hardening: lint.yml pins reviewdog/action-cpplint@master (mutable branch) in a token-holding job

1 participant