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

Deprecate 'java.configuration.checkProjectSettingsExclusions' #2311

Conversation

jdneo
Copy link
Collaborator

@jdneo jdneo commented Feb 8, 2022

The setting java.configuration.checkProjectSettingsExclusions was introduced to mitigate the problem that metadata files are generated at the project root. Now since we have java.import.generatesMetadataFilesAtProjectRoot to completely solve the problem, I think it's a good time to deprecated it.

This PR deprecates java.configuration.checkProjectSettingsExclusions, and change its default value to false, so:

  • If users want to change the place to generate metadata files they can use java.import.generatesMetadataFilesAtProjectRoot.
  • If users just want to hide the metadata files at the file explorer, they can use files.exclude directly. Though I don't think it's a common case (explicitly let the metadata files generate at project root but hide them in the file explorer).

Signed-off-by: Sheng Chen sheche@microsoft.com

- If users want to change the place to generate metadata files they can
  use 'java.import.generatesMetadataFilesAtProjectRoot'.
- If users just want to hide the metadata files at the file explorer,
  they can use 'files.exclude' directly.

Signed-off-by: Sheng Chen <sheche@microsoft.com>
Copy link
Collaborator

@testforstephen testforstephen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@testforstephen testforstephen merged commit 8aadf21 into redhat-developer:master Feb 16, 2022
@testforstephen testforstephen added this to the End February 2022 milestone Feb 16, 2022
@jdneo jdneo deleted the cs/checkProjectSettingsExclusions branch February 16, 2022 02:30
fvclaus pushed a commit to fvclaus/vscode-java that referenced this pull request Feb 18, 2022
…-developer#2311)

- If users want to change the place to generate metadata files they can
  use 'java.import.generatesMetadataFilesAtProjectRoot'.
- If users just want to hide the metadata files at the file explorer,
  they can use 'files.exclude' directly.

Signed-off-by: Sheng Chen <sheche@microsoft.com>
Signed-off-by: Frederik Claus <f.v.claus@googlemail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants