cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From daserge <>
Subject [GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...
Date Tue, 13 Oct 2015 06:43:37 GMT
Github user daserge commented on the pull request:
    > We should remove this:
- I know a lot of downstream tools currently use Error: prefix to determine something is an
error vs not.
    It makes sense in terms of backwards compatibility - removed this trimming. 
    Although we will need to revert those code where we have removed the `Error: ` prefixes
or alternatively concat such a prefix automatically in logger for `error` level if it is missing
in an event message.
    In this case we will still have an excess noise but it should be acceptable as error messages
are uncommon.
    What do you think?

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 enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message