Skip to content

Conversation

@nscuro
Copy link
Member

@nscuro nscuro commented Jun 3, 2025

As per https://github.com/orgs/community/discussions/160793#discussioncomment-13312469

Also bumps all actions to their latest versions, and fixes incorrect tag annotations.

As per https://github.com/orgs/community/discussions/160793#discussioncomment-13312469

Also bumps all actions to their latest versions, and fixes incorrect tag annotations.

Signed-off-by: nscuro <[email protected]>
Copilot AI review requested due to automatic review settings June 3, 2025 13:04
@nscuro nscuro added the dependencies Pull requests that update a dependency file label Jun 3, 2025
@nscuro nscuro merged commit 13eea9b into main Jun 3, 2025
3 checks passed
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

Updates the CI workflow to use the latest versions of various GitHub Actions by pinning specific commits and annotating the correct tags.

  • Pin actions/checkout, setup-java, QEMU, buildx, login, and build-push actions to their latest commit SHAs with tag comments.
  • (Per PR title) Build timeouts remain at 5 minutes and may need adjustment.
Comments suppressed due to low confidence (2)

.github/workflows/ci.yml:21

  • The timeout-minutes is still set to 5, which may cause premature build cancellations. Consider increasing this to 10–20 minutes to accommodate longer-running jobs.
timeout-minutes: 5

.github/workflows/ci.yml:24

  • [nitpick] Pinning to a commit SHA reduces readability and makes future upgrades more manual. Consider using the version tag (actions/[email protected]) directly for clarity and easier maintenance.
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # tag=v4.2.2

@nscuro nscuro deleted the fix-actions branch June 3, 2025 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants