Skip to content

[BRE-2116] Bump actions/checkout to v7.0.1 in non-fork-checkout workflows - #8101

Open
brandonbiete wants to merge 1 commit into
mainfrom
bre-2116/bump-checkout-v7-non-privileged-workflows
Open

[BRE-2116] Bump actions/checkout to v7.0.1 in non-fork-checkout workflows#8101
brandonbiete wants to merge 1 commit into
mainfrom
bre-2116/bump-checkout-v7-non-privileged-workflows

Conversation

@brandonbiete

Copy link
Copy Markdown

🎟️ Tracking

https://bitwarden.atlassian.net/browse/BRE-2116

📔 Objective

Mechanical version bump (v6.0.2 -> v7.0.1, SHA-pinned) across 8 workflows that use plain pull_request/push/schedule triggers with no fork-content checkout. v7's pull_request_target fork-block does not apply to these, so no allow-unsafe-pr-checkout is needed.

📸 Screenshots

Mechanical version bump (v6.0.2 -> v7.0.1, SHA-pinned) across 8 workflows
that use plain pull_request/push/schedule triggers with no fork-content
checkout. v7's pull_request_target fork-block does not apply to these, so
no allow-unsafe-pr-checkout is needed.

Supersedes the checkout portion of Renovate PR #8027 for these files.
build.yml (privileged fork-checkout) is intentionally excluded and handled
in a separate PR with the required allow-unsafe-pr-checkout review.

Part of BRE-2116 checkout v7 remediation.
@brandonbiete
brandonbiete requested a review from a team as a code owner July 30, 2026 18:19
@brandonbiete
brandonbiete requested a review from dereknance July 30, 2026 18:19
@brandonbiete brandonbiete added the t:tech-debt Change Type - Tech debt label Jul 30, 2026
@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.16%. Comparing base (561bf01) to head (ece56b9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8101      +/-   ##
==========================================
+ Coverage   62.71%   67.16%   +4.44%     
==========================================
  Files        2298     2298              
  Lines      100115   100115              
  Branches     9011     9011              
==========================================
+ Hits        62785    67238    +4453     
+ Misses      35147    30597    -4550     
- Partials     2183     2280      +97     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@djsmith85 djsmith85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes are looking good, and similar feedback as given on bitwarden/clients#22157 (review)

I'd expect test.yml to execute on community PRs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:tech-debt Change Type - Tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants