-
Notifications
You must be signed in to change notification settings - Fork 449
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
@resurrect-processes does not work with npm start
#201
Comments
There have been reports of wrongly saved process names for some javascript tools (grunt or gulp, don't remember now). Also, I just checked and I get the same for I think the issue here is the tool ( Here's the fastest way to set a node process name:
Now Anyway, I'd suggest contributing to |
Thanks for the breadcrumbs @bruno- This seems still to be an issue as I see my npm processes saved as :npm in the file, so it's kind of a problem. I am thinking that if we inject the |
This doesn't work. See this process structure:
:( Still persisted in resurrect as |
Other people have solved this, check the docs https://github.com/tmux-plugins/tmux-resurrect/blob/master/docs/restoring_programs.md#nodejs |
Yep I have looked at that section. I don't wanna come across the wrong way, but... yarn isn't npm. Some people may be able to switch to yarn, but it's sort of silly to assume everyone can. But I mostly understand the problem with npm at this point. It's with the code. I did make a PR there (you can see linked above). One thing I def wanted to swing back to mention is that a quick way to work around all of this silliness is to just make your own shell wrapper script to run. For example two of the npm processes I need are |
This includes all positional args (ie, not anything that is a config), except for the third positional arg when running `npm token revoke ...`, since that is also a secret. Makes the `ps` entry a bit more useful than just "npm", so users can at least get some clue what npm is doing, while minimizing the degree to which we leak anything secret. Closes: #1927 Fixes: https://npm.community/t/8400 Related: tmux-plugins/tmux-resurrect#201
This includes all positional args (ie, not anything that is a config), except for the third positional arg when running `npm token revoke ...`, since that is also a secret. Makes the `ps` entry a bit more useful than just "npm", so users can at least get some clue what npm is doing, while minimizing the degree to which we leak anything secret. Closes: #1927 Fixes: https://npm.community/t/8400 Related: tmux-plugins/tmux-resurrect#201 Credit: @isaacs Close: #2154 Reviewed-by: @ruyadorno
This includes all positional args (ie, not anything that is a config), except for the third positional arg when running `npm token revoke ...`, since that is also a secret. Makes the `ps` entry a bit more useful than just "npm", so users can at least get some clue what npm is doing, while minimizing the degree to which we leak anything secret. Closes: #1927 Fixes: https://npm.community/t/8400 Related: tmux-plugins/tmux-resurrect#201 Credit: @isaacs Close: #2154 Reviewed-by: @ruyadorno
I have checked
~/.tmux/resurrect/last
apparentlynpm start
is being saved as:npm
rather than:npm start
Added it manually and tried restoring it works, so the saving mechanism needs fixing
The text was updated successfully, but these errors were encountered: