Skip to content

Fix for #191 slf4j simple-logger for logging#194

Merged
ktoso merged 1 commit into
git-commit-id:masterfrom
TheSnoozer:master
May 21, 2015
Merged

Fix for #191 slf4j simple-logger for logging#194
ktoso merged 1 commit into
git-commit-id:masterfrom
TheSnoozer:master

Conversation

@TheSnoozer

Copy link
Copy Markdown
Collaborator

I'm just created the PR.
Its way too late to actually pull this right in the repo...:-)

I also override some simple-logger's default properties to look like the normal maven log outpout. However those can still be overriden by setting -Dorg.slf4j.simpleLogger.showThreadName=true.

Link to issue: #191

@TheSnoozer TheSnoozer added this to the 2.1.14 milestone May 21, 2015
@TheSnoozer TheSnoozer changed the title Fix for #191 slf4j simple-logger for logging Fix for https://github.com/ktoso/maven-git-commit-id-plugin/issues/191 slf4j simple-logger for logging May 21, 2015
@TheSnoozer TheSnoozer changed the title Fix for https://github.com/ktoso/maven-git-commit-id-plugin/issues/191 slf4j simple-logger for logging Fix for #191 slf4j simple-logger for logging May 21, 2015
@ktoso

ktoso commented May 21, 2015

Copy link
Copy Markdown
Collaborator

Hah nice, you beat me to it :-) Looks good!

ktoso added a commit that referenced this pull request May 21, 2015
Fix for #191 slf4j simple-logger for logging
@ktoso
ktoso merged commit c9f6423 into git-commit-id:master May 21, 2015
@TheSnoozer

Copy link
Copy Markdown
Collaborator Author

@ktoso ,
Do you even sleep? :-D

Thanks for pulling this in!

@ktoso

ktoso commented May 22, 2015

Copy link
Copy Markdown
Collaborator

Far too little 😉 Will retry releasing today (Seems like encoding problems, weird - I'm on a mac)

@Cousnouf

Cousnouf commented Jun 3, 2015

Copy link
Copy Markdown
Contributor

hi @ktoso, hope everything is fine :)

The slf4j problem is correct I see but in the future 2.1.15 release is that right?

Cheers.

@ktoso

ktoso commented Jun 3, 2015

Copy link
Copy Markdown
Collaborator

Yes, I'll roll an update that fixes this problem soon.

@ktoso

ktoso commented Jun 3, 2015

Copy link
Copy Markdown
Collaborator

More details here #198

@Cousnouf

Cousnouf commented Jun 3, 2015

Copy link
Copy Markdown
Contributor

Okay Konrad,

Thank you! ;)

Great work for that cool plugin by the way...

2015-06-03 14:55 GMT+02:00 Konrad Malawski notifications@github.com:

More details here #198
#198


Reply to this email directly or view it on GitHub
#194 (comment)
.

Marc Cappelletti
ch. des Lys 7
CH - 1010 Lausanne

@ktoso

ktoso commented Jun 3, 2015

Copy link
Copy Markdown
Collaborator

Glad you find it useful, cheers!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants