Suggestions for why parallelshell breaks node-notifier?
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 25/100
- Issue type
- Bug
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- javascript, node.js
- Domain
- cli
Research direction
Start by running the minimal package and npm-script example in the issue, comparing the direct notify command with the parallelshell-wrapped command on an available operating system. Read node-notifier issue #109 and parallelshell issue #52 to determine which project owns the hang; done means documenting a reproducible cause and a clear next change or handoff.
Written by the indexing model from the issue text.
Description
https://github.com/mikaelbr/node-notifier/issues/109
From original Issue:
I've been able to get multiple people on multiple operating systems (OSX and Windows) to reproduce some unexpected behavior with the following package:
{
"dependencies": {
"node-notifier": "^4.5.0",
"parallelshell": "^2.0.0"
},
"scripts": {
"works": "npm run notify",
"broken": "parallelshell \"npm run notify\"",
"notify": "notify -t \"Some Title\" -m \"Some Message\" || echo 'failed'"
}
}
For some reason when notify is called behind something run with parallelshell it will just hang. I'd love to try and figure out how I can get this unstuck but I'm not sure if it is a problem with parallelshell or node-notifier. :( Any ideas?
I've also opened an issue on parallelshel here: keithamus/parallelshell#52
- Dominant language
- JavaScript
- Stars
- 153
- Forks
- 16
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from mikaelbr/node-notifier-cli
-
Difficulty 3/5 1-2 days Newbie friendliness 25/100
-
Add --wait optionOpen
Difficulty 3/5 1-2 days Newbie friendliness 20/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
mikaelbr/node-notifier-cli#22 · 1 comment · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
mikaelbr/node-notifier-cli#18 · 3 comments ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
mikaelbr/node-notifier-cli#9 · 1 reaction ·
All issues in mikaelbr/node-notifier-cli
Similar issues
-
factory-active factory-automatic task-bug-reproduction-success task-identify-harness-labels-done task-identify-issue-type-done
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
vercel/ai#22796 · 2 comments ·
Maintainers usually reply within 1 day
-
documentation
Difficulty 2/5 Half a day Newbie friendliness 78/100
facioquo/stock-indicators-dotnet#2300 ·
Maintainers usually reply within 1 day
-
good first issue needs-triage priority: low
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
melodic-software/claude-code-plugins#6982 · 1 comment ·
Maintainers usually reply within 1 day
-
val() setter uses the first control's type for a mixed selectionPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 Half a day Newbie friendliness 85/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 1-3 hours Newbie friendliness 75/100
douglascrockford/Misty#35 ·
Maintainers usually reply within 1 day