TES 0.4.0 Release #118
TES 0.4.0 Release #118
Conversation
* removed any `title: OPTIONAL` annotations that did not make sense * added `required: []` for required parameters * Added schema version, updating minor version since prefix may be a API breaking change Signed-off-by: Angel Pizarro <[email protected]>
Removed `creation_time` from the required list for `tesTask` as it's marked as `readOnly` as per @adamstruck's [comment](#114 (review)).
…-params Removed protobuf-to-swagger conversion artifacts
|
Susheel and Angel think this will be sufficient for the April meeting. |
|
@susheel @delagoya @briandoconnor Hi guys, are you still planning on merging this with master? |
|
LGTM |
|
+1 Also LGTM |
|
There are merge conflicts though. @denis-yuen do you think these changes should be merged? |
There was a problem hiding this comment.
This is just a general cleanup of the artifacts from converting from Protobuf a while back. I think that all of it is not controversial but will defer to an implementer to say that these changes to do not cause unexpected behavior, since I did move from documented requirements on the spec to swagger's formal mechanism on defining constraints (e.g. OUTPUT ONLY --> readOnly: true)
|
@delagoya looks like git was just confused because Assuming |
|
LGTM |
Task Execution Service 0.4.0 Release
Summary:
task_execution.swagger.jsoninto OAPI 2.0 YAMLrequiredandoptionalfields for OAPI request/response schemaBreaking Changes:
task_schema.swagger.jsonupdated toopenapi/task_schema.swagger.yamlrequiredfields are now explicit in the schema.