-
Notifications
You must be signed in to change notification settings - Fork 14
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Major update #4
Major update #4
Conversation
- Show SUCCESS message even when there are excluded warnings. - Show basename instead of the full path. - Updated messages. - Simplified code.
index.js
Outdated
title: util.format('%sType check succeeded', this.titlePrefix), | ||
message: util.format( | ||
'%s%s', | ||
'Not type errors found.', |
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.
Could this be 'No type errors!'
please? It's chirpier 馃榿
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.
Sure. "not" is obviously a typo.
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.
Done
Thanks for the PR! I'm on my phone right now but I'll try and take a proper look when next at a laptop |
FWIW I feel shame I still haven't written tests for this package 馃槰 |
@johnnyreilly At least there is an issues section 馃檪 |
Released with https://github.com/johnnyreilly/fork-ts-checker-notifier-webpack-plugin/releases/tag/v0.5.0 - thanks for your work! |
Baby steps of actually adding tests now 馃槃 |
Awesome ! |
There was no notification when build succeed with warnings and
excludeWarnings
was set totrue
.Full path was unusable as text was truncated and I never saw an actual filename (I'm on macOS).
"Build succeeded" was quite misleading as notification can fire before webpack emits files, especially in first build.
I slightly updated a code to make it more straightforward 馃槈