Skip to content
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

chore(repo): update gh actions version #30561

Merged
merged 1 commit into from
Apr 10, 2025
Merged

Conversation

meeroslav
Copy link
Contributor

This PR bumps the versions of actions/* GH actions to latest v4.

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

@meeroslav meeroslav self-assigned this Apr 1, 2025
@meeroslav meeroslav requested a review from a team as a code owner April 1, 2025 10:29
@meeroslav meeroslav requested a review from AgentEnder April 1, 2025 10:29
Copy link

vercel bot commented Apr 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Apr 1, 2025 10:37am

@meeroslav meeroslav requested review from Copilot and removed request for AgentEnder April 1, 2025 10:29
Copy link

nx-cloud bot commented Apr 1, 2025

View your CI Pipeline Execution ↗ for commit 29f976f.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-ci ✅ Succeeded 38m 27s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 19s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 2m 31s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-01 11:14:15 UTC

Copy link

@Copilot 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 workflows to use the latest v4 versions of various actions.

  • Bumps checkout actions from v3 to v4 and adds a new "filter: tree:0" parameter.
  • Updates cache actions from v3 to v4.
  • Removes the --base and --head parameters from NX commands in the CI configuration.
Comments suppressed due to low confidence (2)

.github/workflows/ci.yml:32

  • Please verify that the new 'filter: tree:0' parameter is supported by actions/checkout@v4 and that its usage is intentional.
filter: tree:0

.github/workflows/ci.yml:76

  • The removal of the '--base' and '--head' parameters in the format:check command may affect the context in which the check runs. Please confirm that this change is intended.
pnpm nx-cloud record -- nx format:check &

@@ -85,7 +85,7 @@ jobs:
pnpm nx run-many -t check-imports check-commit check-lock-files check-codeowners --parallel=1 --no-dte &
pids+=($!)

pnpm nx affected --targets=lint,test,build,e2e,e2e-ci --base=$NX_BASE --head=$NX_HEAD &
pnpm nx affected --targets=lint,test,build,e2e,e2e-ci &
Copy link
Preview

Copilot AI Apr 1, 2025

Choose a reason for hiding this comment

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

The removal of the '--base' and '--head' parameters in the affected command could alter its behavior. Please verify that this modification is aligned with the intended CI logic.

Suggested change
pnpm nx affected --targets=lint,test,build,e2e,e2e-ci &
pnpm nx affected --targets=lint,test,build,e2e,e2e-ci --base=$BASE_SHA --head=$HEAD_SHA &

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

@meeroslav meeroslav merged commit b82f29b into master Apr 10, 2025
13 checks passed
@meeroslav meeroslav deleted the update/ci-gh-versions branch April 10, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants