-
-
Notifications
You must be signed in to change notification settings - Fork 103
feat: npm install -g ttyper #97
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
base: main
Are you sure you want to change the base?
Conversation
|
Hey @max-niederman, could you please review when you're available? |
max-niederman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First off, I just want to apologize for letting this go unreviewed for so long. I've been really busy lately with college applications and some health issues, but I should now have more time to maintain ttyper.
Anyway, I think there's some issue with this implementation. When I install the npm package and try to run ttyper, I get the following error:

As a minimal reproduction, you can get this error by starting an Alpine linux container, installing the npm package, then running npm install -g ttyper and ttyper. I'm not sure why this happens, exactly, but I suspect this is because it copies the runner file into /usr/local/bin, so trying to get the binary using a path relative to the runner script fails.
|
Hi @max-niederman. Welcome back. I'll take a look. |
| "repository": { | ||
| "type": "git", | ||
| "url": "git+https://github.com/Anush008/ttyper.git" | ||
| }, | ||
| "releasesUrl": "https://github.com/Anush008/ttyper/releases", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Update these URLs to https://github.com/max-niederman/ttyper
|
Hi @max-niederman. Could you try |
This PR intends to add
npm install -g ttyperas an installation strategy for Ttyper. Since NPM is the most used package manager, this would be a valuable addition to the project.https://www.npmjs.com/package/ttyper
Here's how this works:
When
npm installis run, NPM executes the install script, i.e.install.js.The
install.jsfile:bindirectory.node_moduleswith the tar and zip libraries, as they are not needed any further.ttyper, to whatever is set as thebinvalue in thepackage.json.binto runrunner.js, which handles the running of our downloaded binary.The added
npm-releaseAction job:package.jsonto be in parity with the release version.README.mdfrom the root dir to be displayed on the package's homepage.We can commit the version bump if we need to, but that would be an extra commit.
A successful run of this workflow can be found here.
@max-niederman, I'll move the NPM package to you and you can configure the
NPM_ACCESS_TOKENsecret for this repository.