Skip to content

chore: mark closed issues as completed and bump actions/checkout from v6 to v7#1934

Merged
josecelano merged 1 commit into
torrust:developfrom
josecelano:chore/clean-closed-issues-and-update-gh-actions
Jun 23, 2026
Merged

chore: mark closed issues as completed and bump actions/checkout from v6 to v7#1934
josecelano merged 1 commit into
torrust:developfrom
josecelano:chore/clean-closed-issues-and-update-gh-actions

Conversation

@josecelano

Copy link
Copy Markdown
Member

Closed issue specs:

Dependabot PR #1920 follow-up:

  • Bumped actions/checkout from v6 to v7 across all 12 workflow files

Pre-merge checks:

  • cargo machete --with-metadata — pass
  • cargo deny check bans — pass
  • linter all — pass
  • cargo test --doc --workspace — pass

… v6 to v7

- Update frontmatter status from 'open' to 'completed' for issues torrust#1924 and torrust#1925
- Update last-updated-utc to 2026-06-23 for both specs
- Bump actions/checkout from v6 to v7 in all 12 workflow files
@josecelano josecelano requested a review from a team as a code owner June 23, 2026 12:28
Copilot AI review requested due to automatic review settings June 23, 2026 12:28
@josecelano josecelano self-assigned this Jun 23, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR performs repo maintenance by marking two issue-spec documents as completed and updating GitHub Actions workflows to use actions/checkout@v7.

Changes:

  • Updated YAML frontmatter for issues #1924 and #1925 (status and last-updated-utc).
  • Bumped actions/checkout from v6 to v7 across CI workflows.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/issues/open/1924-1669-si-30-decouple-rest-api-core-from-udp-internals.md Marks issue spec as completed via frontmatter update.
docs/issues/open/1925-1669-si-31-configure-cargo-deny-for-layer-boundary-enforcement.md Marks issue spec as completed via frontmatter update.
.github/workflows/upload_coverage_pr.yaml Updates checkout action to v7 for the coverage upload workflow.
.github/workflows/testing.yaml Updates checkout action to v7 in testing jobs.
.github/workflows/os-compatibility.yaml Updates checkout action to v7.
.github/workflows/labels.yaml Updates checkout action to v7.
.github/workflows/generate_coverage_pr.yaml Updates checkout action to v7 for coverage generation.
.github/workflows/docs-lint.yaml Updates checkout action to v7 for docs linting.
.github/workflows/deployment.yaml Updates checkout action to v7 in deployment jobs.
.github/workflows/db-compatibility.yaml Updates checkout action to v7 in DB compatibility jobs.
.github/workflows/db-benchmarking.yaml Updates checkout action to v7 in benchmarking jobs.
.github/workflows/coverage.yaml Updates checkout action to v7 for coverage workflow.
.github/workflows/copilot-setup-steps.yml Updates checkout action to v7 for Copilot setup steps workflow.
.github/workflows/container.yaml Updates checkout action to v7 in container jobs.
Comments suppressed due to low confidence (1)

.github/workflows/upload_coverage_pr.yaml:103

  • actions/checkout@v7 changed behavior for workflow_run events: it blocks checking out fork PR commits. This workflow is workflow_run-triggered and checks out the PR head SHA from artifacts, so fork-based PR coverage uploads may start failing after this bump. Consider removing the checkout entirely (if Codecov upload works with just override_commit/override_pr), or checking out a safe ref (e.g., the base repo default branch) instead of the PR head commit, and verify fork PRs still upload coverage.
      - name: Checkout repository
        uses: actions/checkout@v7
        with:
          ref: ${{ steps.parse_previous_artifacts.outputs.override_commit || '' }}
          path: repo_root


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +4 to +9
status: completed
priority: p2
epic: 1669
github-issue: 1924
spec-path: docs/issues/open/1924-1669-si-30-decouple-rest-api-core-from-udp-internals.md
last-updated-utc: 2026-06-22
last-updated-utc: 2026-06-23
Comment on lines +4 to +10
status: completed
priority: p2
github-issue: 1925
spec-path: docs/issues/open/1925-1669-si-31-configure-cargo-deny-for-layer-boundary-enforcement.md
branch: 1925-configure-cargo-deny-for-layer-boundary-enforcement
related-pr: 1932
last-updated-utc: 2026-06-22
last-updated-utc: 2026-06-23
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.51%. Comparing base (9f5fd83) to head (ad647c7).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1934      +/-   ##
===========================================
- Coverage    79.52%   79.51%   -0.01%     
===========================================
  Files          324      324              
  Lines        23137    23137              
  Branches     23137    23137              
===========================================
- Hits         18399    18398       -1     
  Misses        4468     4468              
- Partials       270      271       +1     

☔ 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.

@josecelano

Copy link
Copy Markdown
Member Author

ACK ad647c7

@josecelano josecelano merged commit 9932ccf into torrust:develop Jun 23, 2026
28 checks passed
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.

2 participants