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

#41 - Correctly validate the type of the exceptions caught in try...catch blocks #47

Merged
merged 1 commit into from Mar 3, 2019

Conversation

csoroiu
Copy link
Contributor

@csoroiu csoroiu commented Apr 12, 2018

No description provided.

@csoroiu csoroiu changed the title Issue 41 #41 - Correctly validate the type of the exceptions caught in try...catch blocks Apr 12, 2018
@csoroiu csoroiu force-pushed the issue-41 branch 2 times, most recently from e9bf86c to 9a2ec6a Compare July 25, 2018 19:43
@csoroiu
Copy link
Contributor Author

csoroiu commented Jul 25, 2018

i've rebased the fix for #41 to latest code. The build fails because the maven wrapper plugin was released recently and seems to be built with java 8, thus all the java 7 builds are failing.

@olamy
Copy link
Member

olamy commented Aug 1, 2018

please rebase it was time to get rid of 1.7 support...

@csoroiu
Copy link
Contributor Author

csoroiu commented Aug 1, 2018

I rebased it. The PR passes all the tests.
Thanks

@csoroiu
Copy link
Contributor Author

csoroiu commented Feb 3, 2019

I rebased it again. The PR passes all the tests.

@olamy
Copy link
Member

olamy commented Feb 14, 2019

Too many changes are not related to the subject. Please limit to only your issue.

@csoroiu
Copy link
Contributor Author

csoroiu commented Mar 1, 2019

I left only the changeset addressing #41.
Thanks

@olamy olamy merged commit fabc20f into mojohaus:master Mar 3, 2019
@csoroiu csoroiu deleted the issue-41 branch April 23, 2020 10:26
@olamy olamy added the bug label Dec 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants