-
-
Notifications
You must be signed in to change notification settings - Fork 72
build(deps): bump actions/checkout from 3 to 6 #551
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 6. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Important
Looks good to me! 👍
Reviewed everything up to 48ca126 in 40 seconds. Click for details.
- Reviewed
62lines of code in2files - Skipped
0files when reviewing. - Skipped posting
5draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .github/workflows/build.yml:19
- Draft comment:
Bumped actions/checkout from v3 to v6 here. Verify if the new default behavior (especially around credential persistence) requires any adjustments. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to verify if the new default behavior requires adjustments, which is against the rules. It does not provide a specific suggestion or ask for a specific test to be written. It is also related to a dependency change, which should not be commented on unless it's a recognized issue.
2. .github/workflows/build.yml:52
- Draft comment:
Updated checkout action to v6 in the Android job. Confirm that the repository is checked out correctly for Android builds. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to confirm that the repository is checked out correctly for Android builds after updating the checkout action to v6. This falls under asking the author to confirm or double-check something, which is against the rules.
3. .github/workflows/build.yml:82
- Draft comment:
The checkout step in the code coverage job was bumped to v6. Please ensure that any changes (such as in credential persistence) do not affect coverage artifact access. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to ensure that changes do not affect coverage artifact access, which violates the rule against asking the author to ensure behavior is intended or tested. It does not provide a specific suggestion or point out a specific issue.
4. .github/workflows/lint.yml:14
- Draft comment:
Updated checkout action to v6 for the formatting job. Verify that this version works seamlessly with the linting setup. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%The comment is asking the PR author to verify that the new version of the checkout action works with the linting setup. This falls under asking the author to ensure behavior is intended or tested, which is against the rules.
5. .github/workflows/lint.yml:29
- Draft comment:
Changed checkout action to v6 in the Clippy job. Ensure that the checkout behavior remains consistent with previous expectations for linting. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to ensure that the behavior remains consistent, which violates the rule against asking for confirmation or verification of behavior. It does not provide a specific suggestion or point out a specific issue.
Workflow ID: wflow_lpL8FU5ARmVZKf3M
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
Greptile SummaryUpdated
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant D as Dependabot
participant GH as GitHub Actions
participant R as Runner
D->>GH: Update actions/checkout v3→v6
Note over GH: build.yml workflow
GH->>R: Checkout code (v6)
Note over R: Uses Node.js 24
R->>R: Build & test on ubuntu/windows/macOS
R->>R: Build Android
R->>R: Generate code coverage
Note over GH: lint.yml workflow
GH->>R: Checkout code (v6)
R->>R: Format check (rustfmt)
R->>R: Lint check (clippy)
|
Bumps actions/checkout from 3 to 6.
Release notes
Sourced from actions/checkout's releases.
... (truncated)
Changelog
Sourced from actions/checkout's changelog.
... (truncated)
Commits
8e8c483Clarify v6 README (#2328)033fa0dAdd worktree support for persist-credentials includeIf (#2327)c2d88d3Update all references from v5 and v4 to v6 (#2314)1af3b93update readme/changelog for v6 (#2311)71cf226v6-beta (#2298)069c695Persist creds to a separate file (#2286)ff7abcdUpdate README to include Node.js 24 support details and requirements (#2248)08c6903Prepare v5.0.0 release (#2238)9f26565Update actions checkout to use node 24 (#2226)08eba0bPrepare release v4.3.0 (#2237)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Important
Bump
actions/checkoutfrom v3 to v6 in GitHub Actions workflows for build, Android, coverage, and linting.actions/checkoutfrom v3 to v6 inbuild.ymlandlint.yml.build,build-android,build-coverage-tarpaulin, andlintjobs to useactions/checkout@v6.build-coverage-grcovjob also updated toactions/checkout@v6.This description was created by
for 48ca126. You can customize this summary. It will automatically update as commits are pushed.