Skip to content
This repository was archived by the owner on Dec 19, 2025. It is now read-only.

Feat/notify level#98

Open
staeke wants to merge 5 commits into
wclr:masterfrom
staeke:feat/notify-level
Open

Feat/notify level#98
staeke wants to merge 5 commits into
wclr:masterfrom
staeke:feat/notify-level

Conversation

@staeke

@staeke staeke commented Oct 4, 2019

Copy link
Copy Markdown

Branched off #93

Fixes for specifying notifyLevel to just get errors in OS notifier

In included the fix for notifying on uncaught errors here too, since the code was related

@staeke

staeke commented Oct 4, 2019

Copy link
Copy Markdown
Author

ping @whitecolor - separated this out now

@staeke

staeke commented Oct 24, 2019

Copy link
Copy Markdown
Author

ping @whitecolor - do you have time to look at these PRs please?

@wclr

wclr commented Nov 30, 2019

Copy link
Copy Markdown
Owner

Don't you think that this is not the responsibility of ts-node-dev?

@staeke

staeke commented Dec 29, 2019

Copy link
Copy Markdown
Author

@whitecolor again - sorry for being late on this - don't know why I wasn't getting proper notifications

Well, you could argue that it's not ts-node-dev's thing to integrate an OS notifier, but since that is already in place, I definitely think this is a must have personally

@staeke

staeke commented Jan 27, 2020

Copy link
Copy Markdown
Author

ping @whitecolor

@avevlad

avevlad commented May 17, 2020

Copy link
Copy Markdown

@whitecolor ping!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants