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

Provide mechanism to disable buildship #388

Merged
merged 1 commit into from
Dec 13, 2017

Conversation

snjeza
Copy link
Contributor

@snjeza snjeza commented Dec 13, 2017

package.json Outdated
@@ -111,6 +111,12 @@
"description": "Traces the communication between VS Code and the Java language server.",
"scope": "window"
},
"java.skip.gradleImporter": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

To be consistent with the existing java.import.exclusions preference and others toggle like prefs, we should have java.import.gradle.enabled

@snjeza snjeza force-pushed the issue-257 branch 2 times, most recently from 3260dff to 59c9429 Compare December 13, 2017 17:47
package.json Outdated
"java.import.maven.enabled": {
"type": "boolean",
"default": true,
"description": "Enable/disable the maven importer.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maven

package.json Outdated
"java.import.gradle.enabled": {
"type": "boolean",
"default": true,
"description": "Enable/disable the gradle importer.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Gradle

Copy link
Collaborator

@fbricon fbricon left a comment

Choose a reason for hiding this comment

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

Please update the README.md with the new preferences

Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>
@snjeza
Copy link
Contributor Author

snjeza commented Dec 13, 2017

@fbricon I have updated the PR.

@fbricon fbricon merged commit 36f5930 into redhat-developer:master Dec 13, 2017
@snjeza snjeza deleted the issue-257 branch December 13, 2017 20:40
@fbricon fbricon added this to the Mid December 2017 milestone Dec 15, 2017
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