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.

Sign up for GitHub

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

Jump to bottom

CLN: Move validation of workflow key #590

Merged

tnatt merged 1 commit into equinor:main from tnatt:workflow_format

Apr 10, 2024

Collaborator

tnatt commented Apr 10, 2024

PR to move the validation of the workflow key outside of the FmuProvider, and utilizing existing pydantic model for the validation.


          CLN: Move validation of workflow key

4f9183c

tnatt requested review from mferrera and perolavsvendsen

April 10, 2024 09:56

perolavsvendsen approved these changes

View reviewed changes

Member

perolavsvendsen left a comment

LGTM 🚀 Not a breaking change (still allow dict as input), but deprecate this possibility with a warnings sounds sensible.

tnatt merged commit f0a91df into equinor:main

13 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet