Skip to content

Conversation

@RichardHelm
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d61b119) to head (bfe0452).
⚠️ Report is 1707 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #2652     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files          123       430    +307     
  Lines         1562     10177   +8615     
  Branches       108      1868   +1760     
===========================================
+ Hits          1562     10177   +8615     
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

Copy link

Copilot AI left a comment

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 updates GitHub Actions workflow dependencies to their latest versions and modernizes deprecated workflow syntax to comply with actionlint recommendations, including the addition of actionlint as a linting step for workflow files.

Key changes:

  • Updated GitHub Actions to newer major versions (checkout@v4, codeql-action@v3, stale@v9, etc.)
  • Replaced deprecated ::set-output commands with the modern $GITHUB_OUTPUT syntax
  • Added actionlint workflow validation step

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/stale.yml Updated actions/stale from v4 to v9
.github/workflows/release-please.yml Updated actions/github-script to v7 and migrated set-output syntax
.github/workflows/pre-release.yml Updated checkout to v4, added missing step ID, migrated set-output syntax
.github/workflows/codeql-analysis.yml Updated checkout to v4 and codeql-action to v3
.github/workflows/_unit-test.yml Updated checkout to v4, codecov-action to v4, added missing step ID
.github/workflows/_publish.yml Updated checkout to v4 and added proper quoting for shell arguments
.github/workflows/_lint.yml Updated checkout to v4 and added actionlint validation step
.github/workflows/_deploy-deno.yml Updated checkout to v4
.github/workflows/_check-metadata.yml Updated checkout to v4 and improved GITHUB_OUTPUT variable quoting
.github/workflows/_build.yml Updated checkout to v4

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

@RichardHelm
Copy link
Contributor Author

@copilot Resolve the conflicts with main

Copy link
Contributor

Copilot AI commented Dec 15, 2025

@RichardHelm I've opened a new pull request, #2666, to work on those changes. Once the pull request is ready, I'll request review from you.

@RichardHelm
Copy link
Contributor Author

@copilot Fix the issues identified by actionlint

Copy link
Contributor

Copilot AI commented Dec 15, 2025

@RichardHelm I've opened a new pull request, #2667, to work on those changes. Once the pull request is ready, I'll request review from you.

TaylorJ76
TaylorJ76 previously approved these changes Dec 16, 2025
jhajduk-vonage
jhajduk-vonage previously approved these changes Dec 16, 2025
@RichardHelm RichardHelm merged commit 655d2cf into main Dec 16, 2025
16 checks passed
@RichardHelm RichardHelm deleted the VIV-000-add-actionlint branch December 16, 2025 12:27
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.

4 participants