-
Notifications
You must be signed in to change notification settings - Fork 275
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
Fix error reporting on 2.11 #1177
Conversation
|
||
static { | ||
try { | ||
reporterField = Driver.class.getDeclaredField("reporter"); // NoSuchFieldException |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This field could be maintained to get access to the ReportableMainClass
implementation on our side, but I do think this would be unnecessary added complexity.
@andrefmrocha #1176 is merged. Let's proceed with this one? |
d9d737a
to
c0c9741
Compare
Should be gtg! Let me know what you think |
@andrefmrocha can you fix #1177 (comment) from previous PR and we can merge this one |
Was precisely doing it right now 😄 ! |
Description
Closes #1161 . Changed how the reporter is obtained from the
MainClass
, since the call togetDeclaredField
is obtaining the wrong reporter for scala 2.11.Motivation
Fixes problems regarding building for scala 2.11, where the build was exiting without flagging errors, whenever the
DiagnosticsReporter
was used