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

Added detailed log in case of ReflectionTypeLoadException #7

Closed
wants to merge 1 commit into from

Conversation

th-wyns
Copy link

@th-wyns th-wyns commented Feb 24, 2017

Added the detailed log for ReflectionTypeLoadException in another method for TypeResolver.

One another thing. I think this logging avoids the 'IgnorableException' method call because it catches the error before it. Please check it if it's necessary because if it is then this solution is not good.

@tusmester tusmester self-assigned this Feb 24, 2017
@tusmester
Copy link
Member

Cool, thanks, this is a good idea! I'll make a few modifications ;-) to finalize this.

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.

2 participants