Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LogManager.Shutdown - Verify that active config exists #1937

Merged

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Jan 28, 2017

Small enhancement to #1899, so it doesn't attempt to load config if it wasn't loaded already.


This change is Reviewable

@snakefoot snakefoot force-pushed the LogFactoryDisposeCloseTargets branch from e222551 to 546e625 Compare January 28, 2017 10:49
@codecov-io
Copy link

codecov-io commented Jan 28, 2017

Current coverage is 82% (diff: 82%)

Sunburst

No coverage report found for master at 6b9c2cc.

Powered by Codecov. Last update 6b9c2cc...4966f2a

@snakefoot snakefoot force-pushed the LogFactoryDisposeCloseTargets branch from 546e625 to 4966f2a Compare January 28, 2017 10:55
@304NotModified 304NotModified added this to the 4.4.2 milestone Feb 3, 2017
@304NotModified 304NotModified added the enhancement Improvement on existing feature label Feb 3, 2017
@304NotModified 304NotModified merged commit 30970e9 into NLog:master Feb 3, 2017
@304NotModified
Copy link
Member

👍

@snakefoot snakefoot deleted the LogFactoryDisposeCloseTargets branch October 10, 2017 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement on existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants