Fix for issue #1 #2

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@ronosaurus
Contributor

ronosaurus commented Jan 12, 2013

The %method PatternLayout pattern does not report the correct call site.
It should report the client call site, not the wrapper call site.

I'm new to github...not sure if I included everything you need. Let me know if I did something wrong.

Fix for issue #1
The %method PatternLayout pattern does not report the correct call site.
It should report the client call site, not the wrapper call site.
@ferventcoder

This comment has been minimized.

Show comment
Hide comment
@ferventcoder

ferventcoder Jan 23, 2013

Owner

I would not close the pull request unless you found something is wrong with it. :D

Owner

ferventcoder commented Jan 23, 2013

I would not close the pull request unless you found something is wrong with it. :D

@ferventcoder ferventcoder reopened this Jan 23, 2013

@@ -24,7 +24,7 @@
<foreColor value="Blue, HighIntensity"/>
</mapping>
<layout type="log4net.Layout.PatternLayout">
- <conversionPattern value="%message%newline"/>
+ <conversionPattern value="%logger.%method %-5level %message%newline"/>

This comment has been minimized.

@ferventcoder

ferventcoder Feb 3, 2013

Owner

This one right here is not something I would normally add. The console stays very focused only on the messages this way.

@ferventcoder

ferventcoder Feb 3, 2013

Owner

This one right here is not something I would normally add. The console stays very focused only on the messages this way.

@ferventcoder

This comment has been minimized.

Show comment
Hide comment
@ferventcoder

ferventcoder Apr 4, 2013

Owner

This is in. Thanks so much for your patience

Owner

ferventcoder commented Apr 4, 2013

This is in. Thanks so much for your patience

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