Spring/spring-tools
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Manifest.ym: applications property is no longer considered required.

Browse Source
This commit is contained in:
Kris De Volder
2017-01-23 08:50:29 -08:00
parent 981674faa9
commit 46818d3462
2 changed files with 1 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
vscode-extensions/vscode-manifest-yaml/src/main/java/org/springframework/ide/vscode/manifest/yaml/ManifestYmlSchema.java
View File

@@ -92,8 +92,7 @@ public class ManifestYmlSchema implements YamlSchema {
YType t_env = f.ymap(t_string, t_string);
// define schema structure...
TOPLEVEL_TYPE.addProperty(f.yprop("applications", f.yseq(application))
.isRequired(true));
TOPLEVEL_TYPE.addProperty(f.yprop("applications", f.yseq(application)));
TOPLEVEL_TYPE.addProperty("inherit", t_string, descriptionFor("inherit"));
YTypedPropertyImpl[] props = {

13
vscode-extensions/vscode-manifest-yaml/src/test/java/org/springframework/ide/vscode/manifest/yaml/ManifestYamlEditorTest.java
View File

@@ -737,19 +737,6 @@ public class ManifestYamlEditorTest {
}
@Test public void applicationsPropertyIsRequired() throws Exception {
Editor editor;
//when the file is empty (there is no AST at all)
editor = harness.newEditor(
"buildpack: some-buildpack"
);
editor.assertProblems(
"buildpack: some-buildpack|'applications' is required"
);
}
@Test public void namePropertyIsRequired() throws Exception {
Editor editor = harness.newEditor(
"applications:\n" +