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

Bump actions/checkout to v4.1.7. #17703

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

ScottTodd
Copy link
Member

This should (hopefully) fix warnings about "Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20" in CI logs. For example: https://github.com/iree-org/iree/actions/runs/9575919714
image

We have been using commit hashes instead of tags, since they are more stable and that was a policy recommendation for Google-managed GitHub repos: https://opensource.google/documentation/reference/github/services#actions

When using a third-party action (one not hosted in a Google-managed org), a fixed version of the action MUST be used by specifying a specific commit, rather than a branch like "master", or a tagged release, which can be overwritten by any maintainer of the action. Docker images should always be run at a fixed version rather than "latest".

Using release tags is easier and more common, so switching back to that style.

This should (hopefully) fix warnings about "Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20" in CI logs.

We have been using commit hashes instead of tags, since they are more stable and that was a policy recommendation for Google-managed GitHub repos. Using release tags is easier and more common, so switching back to that style.
@ScottTodd ScottTodd added the infrastructure Relating to build systems, CI, or testing label Jun 19, 2024
@ScottTodd
Copy link
Member Author

Two warnings remain: https://github.com/iree-org/iree/actions/runs/9585451135?pr=17703

[python_release_packages](https://github.com/iree-org/iree/actions/runs/9585451135/job/26431359927)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-python@d27e3f3d7c64b4bbf8e4abfb9b63b83e846e0435. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
[build_test_runtime :: windows-2022](https://github.com/iree-org/iree/actions/runs/9585451135/job/26431359262)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: ilammy/msvc-dev-cmd@7315a94840631165970262a99c72cfb48a65d25d. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

I'll separately go through and bump the other action versions.

@ScottTodd ScottTodd marked this pull request as ready for review June 19, 2024 16:59
@ScottTodd ScottTodd merged commit 1997902 into iree-org:main Jun 19, 2024
54 of 55 checks passed
@ScottTodd ScottTodd deleted the infra-action-versions branch June 19, 2024 17:01
ScottTodd added a commit that referenced this pull request Jun 25, 2024
)

Follow-up to #17703, updating more
actions.

Notably this includes updates to artifacts v4:
https://github.blog/changelog/2023-12-14-github-actions-artifacts-v4-is-now-generally-available/,
which includes substantial performance improvements (e.g. PkgCI "Upload
wheel artifacts" step 26s --> 4s). That is a _breaking change_:
* Artifacts v4 is not cross-compatible with previous versions. For
example, an artifact uploaded using v3 cannot be used with
actions/download-artifact@v4.
LLITCHEV pushed a commit to LLITCHEV/iree that referenced this pull request Jul 30, 2024
This should (hopefully) fix warnings about "Node.js 16 actions are
deprecated. Please update the following actions to use Node.js 20" in CI
logs. For example:
https://github.com/iree-org/iree/actions/runs/9575919714

![image](https://github.com/iree-org/iree/assets/4010439/33699fe2-5a62-4874-8318-2e2d50f2b8e2)

We have been using commit hashes instead of tags, since they are more
stable and that was a policy recommendation for Google-managed GitHub
repos:
https://opensource.google/documentation/reference/github/services#actions
> When using a third-party action (one not hosted in a Google-managed
org), a fixed version of the action MUST be used by [specifying a
specific
commit](https://help.github.com/en/github/automating-your-workflow-with-github-actions/workflow-syntax-for-github-actions#jobsjob_idstepsuses),
rather than a branch like "master", or a tagged release, which can be
overwritten by any maintainer of the action. Docker images should always
be run at a fixed version rather than "latest".

Using release tags is easier and more common, so switching back to that
style.

Signed-off-by: Lubo Litchev <[email protected]>
LLITCHEV pushed a commit to LLITCHEV/iree that referenced this pull request Jul 30, 2024
…e-org#17705)

Follow-up to iree-org#17703, updating more
actions.

Notably this includes updates to artifacts v4:
https://github.blog/changelog/2023-12-14-github-actions-artifacts-v4-is-now-generally-available/,
which includes substantial performance improvements (e.g. PkgCI "Upload
wheel artifacts" step 26s --> 4s). That is a _breaking change_:
* Artifacts v4 is not cross-compatible with previous versions. For
example, an artifact uploaded using v3 cannot be used with
actions/download-artifact@v4.

Signed-off-by: Lubo Litchev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Relating to build systems, CI, or testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants