Github user asfgit closed the pull request at:
https://github.com/apache/cordova-cli/pull/224
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-148435203
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
ena
Github user daserge commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-148376439
@nikhilkh, updated, please take a look.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your proj
Github user stevengill commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147800522
lgtm
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user daserge commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147752515
Ok, thanks, I will update it this way.
I will also change the behavior similar to [@vladimir-kotikov
changes](https://github.com/MSOpenTech/cordova-android/commi
Github user daserge commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/224#discussion_r41882234
--- Diff: src/logger.js ---
@@ -45,44 +46,45 @@ function formatError(error, isVerbose) {
} else {
message = error.message;
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147750643
Yes, you are correct, for errors we could continue to having a prefix.
Removing the prefix from an existing message and adding a prefix might be fine.
---
If your
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/224#discussion_r41881734
--- Diff: src/logger.js ---
@@ -45,44 +46,45 @@ function formatError(error, isVerbose) {
} else {
message = error.message;
Github user daserge commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147620672
> We should remove this:
https://github.com/apache/cordova-cli/pull/224/files#diff-567859a4cc8b910de0454da79959bde6R53
- I know a lot of downstream tools currently
Github user daserge commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/224#discussion_r41831327
--- Diff: src/logger.js ---
@@ -59,13 +58,12 @@ function formatError(error, isVerbose) {
logger.log = function (logLevel, message) {
if (thi
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/224#discussion_r41816178
--- Diff: src/logger.js ---
@@ -59,13 +58,12 @@ function formatError(error, isVerbose) {
logger.log = function (logLevel, message) {
if (th
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147556117
We should remove this:
https://github.com/apache/cordova-cli/pull/224/files#diff-567859a4cc8b910de0454da79959bde6R53
- I know a lot of downstream tools currently u
Github user gorkem commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147553471
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user shazron commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147543581
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enable
Github user TimBarham commented on the pull request:
https://github.com/apache/cordova-cli/pull/224#issuecomment-147536916
:+1: :smile:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this f
GitHub user daserge opened a pull request:
https://github.com/apache/cordova-cli/pull/224
CB-9784 Remove CLI logger levels prefixes
`EventEmitter` usage provides info on an event level so the prefixing may
be superfluous taking into account we have coloring distinction for event
le
16 matches
Mail list logo
