-
Notifications
You must be signed in to change notification settings - Fork 118
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
classpathentry should respect pom resource excludes/includes #1625
base: main
Are you sure you want to change the base?
Conversation
f45c317
to
1e4fa5d
Compare
Test Results 214 files ±0 214 suites ±0 15m 53s ⏱️ - 6m 7s For more details on these failures, see this check. Results for commit 55afb02. ± Comparison against base commit 369b34f. ♻️ This comment has been updated with latest results. |
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.
Hello @qxo, thank you for providing this PR.
In general this change looks reasonable, but we have to work through a few things until it can be considered for submission.
First of all, you have to sign the Eclipse Contributor Agreement, otherwise we are not allowed to accept your change.
Second please add a test-case to org.eclipse.m2e.jdt.tests in order to demonstrate the missing feature and to ensure it will continue to work in the future.
About the change itself, please see the remarks below.
org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java
Outdated
Show resolved
Hide resolved
org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java
Outdated
Show resolved
Hide resolved
1cb6b9e
to
a3a9641
Compare
a3a9641
to
55afb02
Compare
55afb02
to
40a80f4
Compare
excludes/includes,: so we build project on eclipse without org.eclipse.m2e.core.maven2Builder
40a80f4
to
e0994ec
Compare
so we build project on eclipse without org.eclipse.m2e.core.maven2Builder