-
Notifications
You must be signed in to change notification settings - Fork 320
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
Adapt to new Compilation Participant API in Eclipse 4.28 #2636
Conversation
@szarnekow would also be nice if you can give this a try locally |
* | ||
* @author dietrich - Initial contribution and API | ||
*/ | ||
public interface CompilationParticipantExtension { |
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.
any other idea how to fix "missing override annotations" issues?
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 is a really nice backward compatibility mode.
private static final Version VERSION_3_34_0 = new Version(3, 34, 0); | ||
|
||
private static Version installedJdtCoreVersion; | ||
private Set<BuildContext> files; |
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.
revert this or is set ok?
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.
BuildContext does not implement equals or hashCode. Should be fine.
unfortunately we dont seem to have any tests at all for this |
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.
Looks good. I'll try locally a bit later
* | ||
* @author dietrich - Initial contribution and API | ||
*/ | ||
public interface CompilationParticipantExtension { |
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 is a really nice backward compatibility mode.
private static final Version VERSION_3_34_0 = new Version(3, 34, 0); | ||
|
||
private static Version installedJdtCoreVersion; | ||
private Set<BuildContext> files; |
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.
BuildContext does not implement equals or hashCode. Should be fine.
92f5bd9
to
fe62b1e
Compare
workbenchTestHelper.tearDown | ||
} | ||
|
||
@Test def void testIfThereIsAnyStatum() { |
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.
@szarnekow what do you think?
/** | ||
* @author Christian Dietrich - Initial contribution and API | ||
*/ | ||
class QuickDebugSourceInstallingCompilationParticipantTest extends AbstractXtendUITestCase { |
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.
Quick - that's the spirit :)
Fixes #2613 Signed-off-by: Christian Dietrich <christian.dietrich@itemis.de>
fe62b1e
to
743b024
Compare
No description provided.