-
Notifications
You must be signed in to change notification settings - Fork 8
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
test: added testing for the exit handlers #64
Conversation
|
ahhhh the lockfile |
wdym lockfile missing the CI at that commit says but the file is there what is going on |
It's because you're using a different |
alr how about now |
package.json
Outdated
@@ -51,6 +51,7 @@ | |||
"dependencies": { | |||
"@msgpack/msgpack": "3.0.0-beta2", | |||
"debounce": "^2.1.0", | |||
"sinon": "^19.0.2", |
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.
Testing modules should be installed as devDependencies
I know.
It's not perfect.
It's far from perfect.
But this feature should be testable too.