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

Bugfix: Throw configException when registering invalid extension assembly/type. #1553

Merged
merged 4 commits into from
Jul 23, 2016

Conversation

304NotModified
Copy link
Member

Fixes #1508. Supersedes #1517

Jeinhaus and others added 3 commits July 23, 2016 20:13
…gistering an assembly or type with the <extensions> Element (with <nlog throwConfigExceptions='true'>).
Add two new testcases for invalid extension registration.
@304NotModified
Copy link
Member Author

Strange, it works on my local machine

@codecov-io
Copy link

Current coverage is 76% (diff: 88%)

Merging #1553 into master will increase coverage by <1%

@@             master      #1553   diff @@
==========================================
  Files           269        269          
  Lines         16266      16284    +18   
  Methods        2605       2605          
  Messages          0          0          
  Branches       1769       1771     +2   
==========================================
+ Hits          12367      12388    +21   
+ Misses         3487       3482     -5   
- Partials        412        414     +2   

Sunburst

Powered by Codecov. Last update 698ca65...0750c5e

@304NotModified 304NotModified changed the title Bugfix: Throw exception when registering invalid extension assembly/type. Bugfix: Throw configException when registering invalid extension assembly/type. Jul 23, 2016
@304NotModified 304NotModified added this to the 4.3.6 milestone Jul 23, 2016
@304NotModified 304NotModified added the bug Bug report / Bug fix label Jul 23, 2016
@304NotModified 304NotModified merged commit ee98bf2 into master Jul 23, 2016
@304NotModified 304NotModified deleted the pr-1517 branch July 26, 2016 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report / Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants