Skip to content

Comments

Improve javy npm cli wrapper#220

Merged
surma merged 2 commits intomainfrom
surma/npm-update
Feb 1, 2023
Merged

Improve javy npm cli wrapper#220
surma merged 2 commits intomainfrom
surma/npm-update

Conversation

@surma
Copy link
Collaborator

@surma surma commented Jan 31, 2023

After trying our npm cli wrapper for a while, I have arrived at the conclusion that the model I chose originally is annoying for us and annoying for users. This PR updates the script accordingly. These are the highlights of the new behavior:

  • The cli wrapper script's version is independent of the Javy CLI version
  • The cli wrapper automatically grabs the latest release using the GitHub Release API
  • If desired, you can force a particular release by setting FORCE_RELEASE
  • If desired, you can build ToT by setting FROCE_FROM_SOURCE.
  • The script has been refactored so that it can be shared with function-runner without major modifications. In a future step, I might turn this into a module that can be shared.

@surma surma merged commit ee8d3e4 into main Feb 1, 2023
@surma surma deleted the surma/npm-update branch February 1, 2023 15:58
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.

2 participants