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

set node-version output #173

Closed
wants to merge 2 commits into from
Closed

set node-version output #173

wants to merge 2 commits into from

Conversation

panva
Copy link
Contributor

@panva panva commented Jul 17, 2020

This PR adds node-version to the action output.

closes #150

@panva
Copy link
Contributor Author

panva commented Aug 10, 2021

@MaksimZhukov @dmitry-shibanov is this something you may look into?

@samhed
Copy link

samhed commented Oct 22, 2021

This would be nice!

setup-node@v1 had something like this:

Run actions/setup-node@v1
  with:
    always-auth: false
    node-version: 10.x
/opt/hostedtoolcache/node/10.24.1/x64/bin/node --version
v10.24.1
/opt/hostedtoolcache/node/10.24.1/x64/bin/npm --version
6.14.12

But now with @v2 we only get:

Run actions/setup-node@v2
  with:
    always-auth: false
    check-latest: false
    token: ***

samhed referenced this pull request in novnc/noVNC Oct 22, 2021
Apparently the new eslint version doesn't work with the older version of
node that we were using. Asking for '@v2' seems to help. It's unclear
what version of node we get now though since v2 isn't as verbose in its
output.
@btkostner
Copy link

Can we get some movement on reviewing this PR? This output is needed to make a better cache key (invalidate when minor version increases.)

@LinusU
Copy link

LinusU commented Nov 25, 2021

ping @0xdeafcafe @dmitry-shibanov, would any of you be able to review/merge this? ☺️

@0xdeafcafe
Copy link
Contributor

I would love to merge, but I'm not a project owner @LinusU 😅

@panva
Copy link
Contributor Author

panva commented Nov 27, 2021

@dmitry-shibanov @MaksimZhukov going by the upvotes this is a sought after feature.

Copy link

@calbearox calbearox left a comment

Choose a reason for hiding this comment

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

@LinusU
Copy link

LinusU commented Feb 25, 2022

ping @dmitry-shibanov, any chance of landing this?

@panva
Copy link
Contributor Author

panva commented Jun 30, 2022

Coming up on 2 years now with no feedback what so ever, please land it, close it, or ask for changes. 🙏

cc @dmitry-shibanov @marko-zivic-93 @vsafonkin

@panva panva mentioned this pull request Jun 30, 2022
2 tasks
@panva panva closed this Jul 4, 2022
@panva
Copy link
Contributor Author

panva commented Jul 4, 2022

Superseded by #534

deining pushed a commit to deining/setup-node that referenced this pull request Nov 9, 2023
Bumps [eslint](https://github.com/eslint/eslint) from 7.23.0 to 7.24.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.23.0...v7.24.0)

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

Provide a output for the selected Node.js version
8 participants