-
Notifications
You must be signed in to change notification settings - Fork 27
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 includes of 3rd party libraries from the org.eclipse.jdt feature #107
Remove includes of 3rd party libraries from the org.eclipse.jdt feature #107
Conversation
57f5885
to
f3d9867
Compare
I would rather see this one delayed for next release to reduce possible "surprises". |
Wow, the verification build doesn't really test anything? |
For this repo yes. |
As requested in #106 (comment) I created a PR, but given this verification build doesn't really test much of anything and given we are post m3, it's probably not a good time to do potentially disruptive changes. |
We can try revisit this as soon as 4.34 / 2024-12 opens for development. |
f3d9867
to
e210206
Compare
FYI, this related change has been merged: |
No description provided.