-
Notifications
You must be signed in to change notification settings - Fork 401
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 bad .git resourceFilter #2251
Conversation
re-test this please. |
The default value for the client settings 'java.project.resourceFilters' needs to be updated as well. |
re-test this please. |
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.
Ok there's a small bug here slightly related.
https://github.com/eclipse/eclipse.jdt.ls/blob/d65385a0d09df1c3a904a6644efe1b0c1a7860d0/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/preferences/Preferences.java#L1042 never passes in the language server default value, so if the client doesn't set it, it doesn't fall back to https://github.com/eclipse/eclipse.jdt.ls/blob/d65385a0d09df1c3a904a6644efe1b0c1a7860d0/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/preferences/Preferences.java#L628 . Just need to pass that value as the 3rd argument to getList(..)
Fixes eclipse-jdtls#2244 Signed-off-by: Fred Bricon <[email protected]>
Signed-off-by: Fred Bricon <[email protected]>
done |
Fix bad .git resourceFilter (fixes #2244)
also fixes wrong jdt compiler settings (don't squash commits)