Skip to content
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

refactor: workflow type/version validation #40

Open
uniqueg opened this issue Jun 20, 2022 · 0 comments
Open

refactor: workflow type/version validation #40

uniqueg opened this issue Jun 20, 2022 · 0 comments

Comments

@uniqueg
Copy link
Member

uniqueg commented Jun 20, 2022

Validating incoming workflow run requests against the currently supported worklow types/versions invoked a call to the GET /service-info endpoint to get the latest info on supported types/versions. However, the way this is currently implemented bears the risk of circular imports.

In particular, it would be nice to get rid of the dedicated pro_wes.ga4gh.wes.states module in favor of the State enum in pro_wes.ga4gh.wes.models, but this currently causes a circular import. Attempt to refactor such that no circular import arises.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant