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

Allow using .tool-versions for node-version-file #445

Closed
wants to merge 1 commit into from

Conversation

deiga
Copy link

@deiga deiga commented Mar 15, 2022

Description:
Add parsing of .tool-versions file format.
This enables developers which use asdf to use their version file to
steer which version of node to install.
This change should be fully backwards compatible as it only does things
if a certain string is found in the node-version-file

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

This enables developers which use `asdf` to use their version file

Signed-off-by: Timo Sand <[email protected]>
@deiga deiga requested a review from a team March 15, 2022 20:46
@brcrista
Copy link
Contributor

Hey @deiga, thanks for the PR. This is my first time hearing about asdf, but I think the approach makes sense. If I'm understanding correctly, you don't actually have to install asdf on the build machine; this is just adding support for its syntax. Is that right?

Since this is a new feature, if you're able to put together a minimal workflow that uses deiga/setup-node, that would go a long way towards helping us see how it works.

@runlevel5
Copy link

Thanks for the PR, we are also using asdf for our projects (due to mix-tech nature). Love to see this PR going in

@IvanZosimov
Copy link
Contributor

@deiga, 👋 Just a gentle ping 💌

@dmitry-shibanov
Copy link
Contributor

Hello @deiga. Thank you for your pull request. For now I'm going to close the pull request because the logic was implemented in this pull request. If you have any concerns feel free to ping us.

@deiga deiga deleted the enable-asdf-node-version-file branch September 8, 2022 17:57
deining pushed a commit to deining/setup-node that referenced this pull request Nov 9, 2023
…ions#445)

Bumps [all-contributors-cli](https://github.com/all-contributors/all-contributors-cli) from 6.23.1 to 6.24.0.
- [Release notes](https://github.com/all-contributors/all-contributors-cli/releases)
- [Commits](all-contributors/cli@v6.23.1...v6.24.0)

---
updated-dependencies:
- dependency-name: all-contributors-cli
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
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.

5 participants