search

jenkinsci / scm-sync-configuration-plugin

Jenkins scm-sync-configuration plugin
https://plugins.jenkins.io/scm-sync-configuration/
MIT License
115 stars 123 forks source link

Reduce SCM Sync logs #7

Closed jcarsique closed 12 years ago

jcarsique commented 12 years ago

See https://issues.jenkins-ci.org/browse/JENKINS-15266

(sorry for the formatting increasing the diff but you used tabs instead of spaces) The changes are on four LOGGER.info reduced to LOGGER.fine.

buildhive commented 12 years ago

Jenkins » scm-sync-configuration-plugin #21 FAILURE Looks like there's a problem with this pull request (what's this?)

fcamblor commented 12 years ago

debug() doesn't exist in JUL. You should rather use finest().

If you can remove the noise by not replacing tabs by spaces (yeah I know, this is bad ;)), it would be great :)

jcarsique commented 12 years ago

Fixed with fine(), sorry :)

buildhive commented 12 years ago

Jenkins » scm-sync-configuration-plugin #22 SUCCESS This pull request looks good (what's this?)

jcarsique commented 12 years ago

Redone in two commits to remove the noise.

buildhive commented 12 years ago

Jenkins » scm-sync-configuration-plugin #23 SUCCESS This pull request looks good (what's this?)

fcamblor commented 12 years ago

merged, thanks :)