-
Notifications
You must be signed in to change notification settings - Fork 56
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
adding check to make sure d exists before trimming, #1086
Conversation
stops spinner while interactivity initing to keep it from loading over text
1 similar comment
@@ -49,6 +49,9 @@ exportables.buildJSON = (npmConfig) => { | |||
promzardConfig = path.join(resources, 'init-config.js'); | |||
} | |||
|
|||
// Stop spinner from being in our way while entering package.json data | |||
log.spinner.stop(); |
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.
Nice
Since the init operations still have relatively low coverage, I don't think the coverage decrease should block this. There is also no clear way to add a simple test for this (which is actually due to the previous statement). |
Sweet. Thank you. I thought I left a comment asking if there was any test to add but I guess I never added it. Thanks again. |
No worries, I'm going to write some basic tests for this and push them up shortly. This way you can observe and have a good idea for next time :) |
Oh, actually... I have a better idea. I will add basic tests that should've existed and then you can make a PR that adds assertions for the changes! |
sounds great to me :) |
stops spinner while interactivity initing to keep it from loading over text
fixes #771