Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Log 'Verifying' messages at INFO level #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cbeams
Copy link

@cbeams cbeams commented Mar 20, 2018

Problem: WitnessPlugin used println to print "Verifying ..." messages
forcing the output to show up at the console under all conditions.

Solution: Print the same messages using project.logger at INFO level,
such that "Verifying ..." messages are not printed at all by default,
but are printed if -i/--info is provided at the command line.

Problem: WitnessPlugin used `println` to print "Verifying ..." messages
forcing the output to show up at the console under all conditions.

Solution: Print the same messages using `project.logger` at INFO level,
such that "Verifying ..." messages are not printed at all by default,
but are printed if `-i/--info` is provided at the command line.
cbeams added a commit to cbeams/bisq that referenced this pull request Mar 21, 2018
This performs the same function that the maven-enforcer-plugin did in
the pom removed by the previous commit, and also includes a more
comprehensive / up-to-date set of dependencies.

Note that the gradle-witness jar checked in here is one built from the
pull request in signalapp/gradle-witness#26.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant