-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Modified the injection framework to use String for the workspace temp… #11792
Modified the injection framework to use String for the workspace temp… #11792
Conversation
…late instead of the enum
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
public String getTemplateFileName() { | ||
return templateFileName; | ||
} | ||
public WorkspaceTemplate() {} |
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.
In constant class constructor is usually private.
@@ -80,7 +79,7 @@ | |||
}; | |||
|
|||
/** It is Map[node-name, Pair[tab-name, line-number]] */ | |||
private static final Map<String, Pair<String, Integer>> PROJECT_SYMBOL_EXPECTED_TEXT = | |||
private static final ImmutableMap<String, Pair<String, Integer>> PROJECT_SYMBOL_EXPECTED_TEXT = |
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.
Good catch!
@@ -163,7 +163,7 @@ public void importProject( | |||
format( | |||
Resources.toString(getResource("projects/jdt-ls-project-files/project"), UTF_8), | |||
projectName) | |||
.getBytes()); | |||
.getBytes(UTF_8)); |
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.
Good catch!
Signed-off-by: Dmytro Nochevnov <dnochevnov@codenvy.com>
…late instead of the enum
What does this PR do?
Changes template referencing to String instead of locked down enum
What issues does this PR fix or reference?
https://app.zenhub.com/workspace/o/redhat-developer/che-functional-tests/issues/294
Release Notes
Docs PR