Closed biharck closed 6 years ago
The default value for the origin attribute has been set to None. Also, the xsd schema has been updated removing the requirement. And condiction was added in order to verify the nullable value.
Versions lower than 1.8.2 don't allow the origin attribute within fetchArtifact element. The changes made on https://github.com/gocd-contrib/gomatic/commit/5274606037d3dff91c16952548a20351c9928b37 are breaking Gomatic due to Attribute ‘origin’ is not allowed to appear in element ‘fetchartifact’.
So, the changes are just to guarantee Gomatic to still work on GoCD as 1.8.2.
Somehow our integration tests didn't catch this, it would be nice to understand why
Released as 0.6.3
What
The default value for the origin attribute has been set to None. Also, the xsd schema has been updated removing the requirement. And condiction was added in order to verify the nullable value.
Why
Versions lower than 1.8.2 don't allow the origin attribute within fetchArtifact element. The changes made on https://github.com/gocd-contrib/gomatic/commit/5274606037d3dff91c16952548a20351c9928b37 are breaking Gomatic due to Attribute ‘origin’ is not allowed to appear in element ‘fetchartifact’.
So, the changes are just to guarantee Gomatic to still work on GoCD as 1.8.2.