Repository navigation
chore(ci): bump actions/checkout to v4 in three workflows - #11454
pig-tongzhi wants to merge 3 commits into
Conversation
RockteMQ-AI
left a comment
There was a problem hiding this comment.
LGTM. Trivial change — bumping actions/checkout to v4 across three workflows. Good housekeeping to keep CI actions on current major versions and pin refs instead of @master.
Automated review by github-manager-bot
|
Issue Evaluation Category: Thank you for identifying the version inconsistency across workflows. Bumping
Assessment: Low-risk, well-scoped change. The 3-file diff is easy to review. The decision to defer the remaining workflows ( Note: This issue requires maintainer action — it involves modifying Automated evaluation by RockteMQ-AI |
What
Bump
actions/checkouttov4in three workflows (3 lines changed):.github/workflows/integration-test.ymlactions/checkout@v2.github/workflows/coverage.ymlactions/checkout@master.github/workflows/codeql_analysis.ymlactions/checkout@v3Why
v4is the current major version, and the other workflows in this repository (e.g.bazel.yml) already use it.@masterincoverage.ymlis an unpinned ref: the action can change without any change in this repository.@v4pins the major version like the rest of the workflows.Scope
This PR intentionally keeps the diff to three files so it is quick to review. I left the remaining workflows (
maven.yaml,license-checker.yaml,misspell_check.yml,snapshot-automation.yml) alone for now; happy to update them in a follow-up PR if you would like that.Thanks for maintaining RocketMQ!