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

Remove forked jdt projects #12050

Merged
merged 1 commit into from
Nov 27, 2018
Merged

Conversation

tsmaeder
Copy link
Contributor

What does this PR do?

Deletes forked code from Eclipse jdt which is not in use since we've moved to jdt.ls

What issues does this PR fix or reference?

Remove JDT projects #11926

@tsmaeder tsmaeder added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. target/che6 labels Nov 26, 2018
@skabashnyuk
Copy link
Contributor

@tsmaeder is this eclipse-che/che-lib#108 something that you are going to merge in master too?

@tsmaeder
Copy link
Contributor Author

@tsmaeder is this eclipse/che-lib#108 something that you are going to merge in master too?

If I ever have time, I might get back to that one, but let's stick to discussing this PR here.

Signed-off-by: Thomas Mäder <[email protected]>
@skabashnyuk
Copy link
Contributor

If I ever have time

What if not?

Am I understand correctly. If after this pr these modules are not used we can remove it?

  • org-eclipse-jdt-core-repack
  • antlr-java5-grammar

@tsmaeder tsmaeder merged commit f4f2829 into eclipse-che:master Nov 27, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Nov 27, 2018
@benoitf benoitf added this to the 6.15.0 milestone Nov 27, 2018
@garagatyi
Copy link

Love this kind of PRs!
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants