-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
build(aio): make less verbose tools #16687
build(aio): make less verbose tools #16687
Conversation
aio/tools/plunker-builder/builder.js
Outdated
@@ -232,10 +232,10 @@ class PlunkerBuilder { | |||
// console.log('** Substituted "' + webExtras + '" for "' + extras + '".'); | |||
return fs.readFileSync(webExtras, 'utf-8'); | |||
} else if (this._existsSync(extras)){ | |||
console.log('** WARNING: no "' + webExtras + '" replacement for "' + extras + '".'); | |||
// console.log('** WARNING: no "' + webExtras + '" replacement for "' + extras + '".'); |
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.
let's keep the warnings
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.
@petebacondarwin, why?
@Foxandxss, what are these warnings for? Are missing something we need to fix or is it normal for some of them to be missing?
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.
Exactly. If they are not actionable then they are just informative and can go.
a19cc2e
to
cc5dec2
Compare
Done. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
@petebacondarwin asked for the tools to be less verbose somewhere (is there an issue for that?)
I left some as a comment because I plan on revisit them.