Skip to content

Updated github action versions#3123

Open
agsaru wants to merge 6 commits intoNetflix:masterfrom
agsaru:action-update
Open

Updated github action versions#3123
agsaru wants to merge 6 commits intoNetflix:masterfrom
agsaru:action-update

Conversation

@agsaru
Copy link
Copy Markdown
Contributor

@agsaru agsaru commented Apr 16, 2026

Summary

Updated github runner to use ubuntu-22.04 instead of ubuntu-latest for stabilty and code consistency.
Used latest github action stable version commits inplace of older versions of actions.

Tests

  • Unit tests added/updated
  • Reproduction script provided (required for Core Runtime)
  • CI passes
  • If tests are impractical: explain why below and provide manual evidence above

Non-Goals

AI Tool Usage

  • No AI tools were used in this contribution
  • AI tools were used (describe below)

Copilot AI review requested due to automatic review settings April 16, 2026 19:23
@agsaru agsaru changed the title Action update Updated github action versions Apr 16, 2026
Copy link
Copy Markdown
Contributor

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

Updates GitHub Actions workflow dependencies to improve CI stability and consistency by pinning the runner OS and action versions.

Changes:

  • Switch workflow jobs from ubuntu-latest to ubuntu-22.04.
  • Pin commonly used GitHub Actions (checkout/setup-python/cache/artifacts/download-artifact/test-reporter/codecov) to specific commit SHAs.

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

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 16, 2026

Greptile Summary

This PR pins all GitHub Action references to full commit SHAs (replacing floating tags like @v4) and switches all three jobs from ubuntu-latest to ubuntu-22.04. Several actions receive major version bumps (e.g. actions/checkout v4→v6, actions/upload-artifact v4→v7, actions/download-artifact v4→v8, dorny/test-reporter v1→v3), which the author has verified via CI.

Confidence Score: 5/5

Safe to merge — a routine CI maintenance update with CI confirmed passing.

All changes are mechanical action-version and runner updates. Previous reviewer concerns were addressed. No logic changes, no security issues, and CI passes per the PR checklist.

No files require special attention.

Important Files Changed

Filename Overview
.github/actions/run-ux-tests/action.yml Pins all action references to full commit SHAs with version comments; major bumps include checkout v4→v6, upload-artifact v4→v7, codecov v5→v6, and setup-miniconda to a pinned SHA.
.github/workflows/ux-tests.yml Replaces ubuntu-latest with ubuntu-22.04 across all three jobs and pins all action SHAs; notably upload-artifact is pinned to v7.0.1 while download-artifact is pinned to v8.0.1 in the same workflow.

Reviews (4): Last reviewed commit: "Merge branch 'master' into action-update" | Re-trigger Greptile

Comment thread .github/workflows/ux-tests.yml
Comment thread .github/workflows/ux-tests.yml Outdated
@romain-intel romain-intel requested a review from npow April 16, 2026 20:09
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