elixir-cloud-aai / proTES

Proxy service for injecting middleware into GA4GH TES requests
Apache License 2.0
5 stars 6 forks source link

feat: update TES models to v1.1 #179

Closed athith-g closed 1 month ago

athith-g commented 2 months ago

Details

Added fields that were added in TES v1.1 to models.py

TES v1.1 changes:

I removed the unnecessary newlines but I'm not sure how to achieve the whitespacing in the existing descriptions.

Summary by Sourcery

Updated TES models to v1.1 by adding new fields and states to support the latest features and configurations as per the TES v1.1 specification.

sourcery-ai[bot] commented 2 months ago

Reviewer's Guide by Sourcery

This pull request updates the TES models to version 1.1 by adding new fields and modifying existing ones in models.py. The changes include adding new fields such as ignore_error, streamable, backend_parameters, and backend_parameters_strict, making the type field optional in TesInput and TesOutput, and adding new states PREEMPTED and CANCELING to TesState.

File-Level Changes

Files Changes
pro_tes/ga4gh/tes/models.py Updated TES models to version 1.1 by adding new fields and modifying existing ones to align with the latest TES schema changes.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - You can change your review settings at any time by accessing your [dashboard](https://sourcery.ai/dashboard): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.
uniqueg commented 2 months ago

Also - tests fail...

athith-g commented 1 month ago

I think the rest of the linting and integration tests will pass once #178 is merged.

uniqueg commented 1 month ago

LGTM, @athith-g - I think you can merge :)