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

Can the release of version node20 be considered? #953

Closed
2 tasks done
178inaba opened this issue Jan 22, 2024 · 10 comments
Closed
2 tasks done

Can the release of version node20 be considered? #953

178inaba opened this issue Jan 22, 2024 · 10 comments
Labels
question Further information is requested

Comments

@178inaba
Copy link
Contributor

Welcome

  • Yes, I understand that the GitHub action repository is not the repository of golangci-lint itself.
  • Yes, I've searched similar issues on GitHub and didn't find any.

Your feature request related to a problem? Please describe.

I have received the following warning.

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Describe the solution you'd like.

I would like a version released using node20 to address the warning.

Describe alternatives you've considered.

None.

Additional context.

v3.7.0 (latest version): node16

using: "node16"

master: node20

using: "node20"

@azisaka
Copy link

azisaka commented Jan 25, 2024

while we don't get a new version you can replace uses: golangci/golangci-lint-action@v3 by uses: golangci/golangci-lint-action@master

@178inaba
Copy link
Contributor Author

@azisaka
Thank you for the reply! I also considered that approach, but it seems that specifying branches is not recommended in the Actions documentation.

Using the default branch of an action may be convenient, but if someone releases a new major version with a breaking change, your workflow could break.
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsuses

Under this release management approach, users should not be referencing an action's default branch, as it's likely to contain the latest code and consequently might be unstable. Instead, you can recommend that your users specify a major version when using your action, and only direct them to a more specific version if they encounter issues.
https://docs.github.com/en/actions/creating-actions/about-custom-actions#good-practices-for-release-management

@rhussmann
Copy link

@178inaba one more workaround that doesn't depend on a moving ref

uses: golangci/golangci-lint-action@c67416616c29c3c48d26b59c45cadb56966d80aa

where c67416616c29c3c48d26b59c45cadb56966d80aa is the full commit SHA where action.yml was updated to point to node20.

Still looking forward to seeing a persistent tag published with this support as I'd prefer not to have to rely on a specific commit.

mdelapenya added a commit to testcontainers/moby-ryuk that referenced this issue Feb 5, 2024
mdelapenya added a commit to testcontainers/moby-ryuk that referenced this issue Feb 5, 2024
…o 25.0.2+incompatible (#102)

* chore(deps): bump github.com/docker/docker

Bumps [github.com/docker/docker](https://github.com/docker/docker) from 24.0.7+incompatible to 25.0.2+incompatible.
- [Release notes](https://github.com/docker/docker/releases)
- [Commits](moby/moby@v24.0.7...v25.0.2)

---
updated-dependencies:
- dependency-name: github.com/docker/docker
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: use non deprecated types

* chore: bump Go version in golangci-lint action

* chore: bump golangci action to run in Node20

See golangci/golangci-lint-action#953

* chore: bump golangci version

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
Co-authored-by: Manuel de la Peña <social.mdelapenya@gmail.com>
@ldez
Copy link
Member

ldez commented Feb 8, 2024

Hello,

I will create a new release in a few minutes.

@ldez ldez added the question Further information is requested label Feb 8, 2024
@ldez
Copy link
Member

ldez commented Feb 8, 2024

Done https://github.com/golangci/golangci-lint-action/tags
v3 updated and v3.7.1 created.

@ldez ldez closed this as completed Feb 8, 2024
@178inaba
Copy link
Contributor Author

178inaba commented Feb 9, 2024

@ldez Thanks!

@lukestoward
Copy link

lukestoward commented Feb 9, 2024

@ldez ideally this change should have been a new major version aligning with how other standard actions managed the upgrade. See actions/cache, actions/setup-go etc.

This has broken our pipeline as a result, as Node 20 requires newer GLIBC versions. Easily fixed, but we will have to pin the version to v3.7.0 now instead of using @v3.

@ldez
Copy link
Member

ldez commented Feb 9, 2024

Sorry, I will create a v4 and try to restore the previous v3.

@ldez
Copy link
Member

ldez commented Feb 9, 2024

I moved the tag v3 to v3.7.0 and created v4 and v4.0.0

I'll keep v3.7.1 to avoid being too breaking.

@lukestoward
Copy link

Great, thanks for the quick resolution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants