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

[readme] add nvshim tool alongside avn #2193

Merged
merged 1 commit into from
Jun 30, 2023
Merged

[readme] add nvshim tool alongside avn #2193

merged 1 commit into from
Jun 30, 2023

Conversation

iamogbz
Copy link
Contributor

@iamogbz iamogbz commented Apr 15, 2020

Made a tool, nvshim, for shimming node binaries using nvm. Useful as an option for people that do not want to add auto loading script to their shell.

Unsure of the community rules about self promotion, just thought to share since it's been working for myself and a couple others.

Copy link
Contributor

@sladyn98 sladyn98 left a comment

Choose a reason for hiding this comment

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

LGTM

README.md Outdated Show resolved Hide resolved
@sladyn98
Copy link
Contributor

sladyn98 commented Apr 5, 2021

I guess this would need to be on hold till the caveats are resolved and merged in master. Would be good to have an waiting-on-author label here : )

@ljharb ljharb added needs followup We need some info or action from whoever filed this issue/PR. and removed needs followup We need some info or action from whoever filed this issue/PR. labels Apr 5, 2021
@ljharb ljharb marked this pull request as draft April 5, 2021 15:03
@iamogbz
Copy link
Contributor Author

iamogbz commented Jun 28, 2023

@sladyn98 @ljharb updated the documentation here and on nvshim caveats to indicate specifically what gets shimmed.

@iamogbz iamogbz marked this pull request as ready for review June 28, 2023 14:58
README.md Outdated Show resolved Hide resolved
@ljharb ljharb changed the title Update example shell integration to show shimming [readme] add nvshim tool alongside avn Jun 30, 2023
@ljharb ljharb merged commit f86847f into nvm-sh:master Jun 30, 2023
@iamogbz iamogbz deleted the patch-1 branch June 30, 2023 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants