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

ci: use actions/checkout@v4 and actions/setup-python@v5 #1717

Merged
merged 5 commits into from
Oct 31, 2024

Conversation

romanc
Copy link
Contributor

@romanc romanc commented Oct 29, 2024

Description

This PR updates the GitHub Actions (GHA) workflows to use actions/checkout@v4 instead of v3 (or v2 in the cartesian case) and actions/setup-python@v5. Development for actions/checkout@v3 and actions/setup-python@v4 stopped ~1 year ago and GH is currently enforcing newer node versions than the one that these actions were designed with, leading to the following warnings

image
warnings in next workflows

image
warnings in cartesian workflows

deploy_release action was following actions/checkout@master. Was this on purpose? Happy to revert if so. Unless there's a good reason, I suggest to keep all actions pinned at ideally the same major version.

pre-commit/action was updated to keep its dependencies up to date and avoid transitive warnings similar to the ones above.

No changes made to currently disabled workflows, i.e. the ones under .github/workflows/_disabled/.

Parent: GEOS-ESM/SMT-Nebulae#89

Requirements

  • All fixes and/or new features come with corresponding tests.
    CI is running as before - just with fewer warnings.
  • Important design decisions have been documented in the approriate ADR inside the docs/development/ADRs/ folder.
    N/A

Roman Cattaneo added 3 commits October 29, 2024 09:56
actions/checkout@v2 received that last "backport fix" in Dec 2022.
Meanwhile GH is even forcing a newer node version (16 instead of 12)
than what v2 was designed with.
@romanc romanc changed the title ci: use actions/checkout@v4 ci: use actions/checkout@v4 and actions/setup-python@v5 Oct 29, 2024
@romanc romanc marked this pull request as ready for review October 29, 2024 10:29
@romanc
Copy link
Contributor Author

romanc commented Oct 29, 2024

@twicki / @FlorianDeconinck continuing the quest to reduce warnings ...

@egparedes / @havogt please review or re-distribute to whoever is knowledgeable / in charge of CI for the next version of gt4py

As far as I can tell, we should be good again for now. We might want to look into automatic reporting of outdated GHA versions in the future.

@romanc
Copy link
Contributor Author

romanc commented Oct 29, 2024

We should also be aware the pre-commit/action is in maintenance mode only (see top of their README file). I wouldn't be very concerned. If we are happy with what we have now, it's super straight forward to just implement ourselves (instead of using their composite action) in case it ever drops support completely.

@egparedes
Copy link
Contributor

cscs-ci run

@egparedes
Copy link
Contributor

cscs-ci run

Copy link
Contributor

@egparedes egparedes left a comment

Choose a reason for hiding this comment

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

LGTM

@egparedes egparedes requested a review from havogt October 30, 2024 12:32
@egparedes egparedes merged commit f1c9d83 into GridTools:main Oct 31, 2024
31 checks passed
@romanc romanc deleted the romanc/gha-checkout-v4 branch October 31, 2024 10:11
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.

3 participants