Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Revise logging severities #1586

Description

@WonderCsabo

We are logging lots of things with WARNINGpriority, which maybe not reasonable. For example we log the found manifest and content; claimed annotations and elements; and generated classes with WARNING log level. I think this should be definitely not WARNING (but INFO), since there is nothing to warn about, these are happening always normally.

Activity

  1. WonderCsabo commented on Oct 10, 2015

    @WonderCsabo
    MemberAuthor

    Actually we used bad name for the log level in the functional test projects... 0a23034 Since there was no log level with such name, we fall back to the default, DEBUG. However, i do not think it is fair to pollute the clients' terminal with debug messages by default. I opened #1588 for that. @yDelouis WDYT?

  2. yDelouis commented on Oct 29, 2015

    @yDelouis
    Contributor

    I agree and merged your PR.
    Please, provide a new PR to fix the code where we used wrong log levels.

  3. WonderCsabo commented on Oct 29, 2015

    @WonderCsabo
    MemberAuthor

    It turned out we log with appropriate severities, the problem was 0a23034.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions