Fixes usage of lists in yaml analysis config

edubarrosTNO commented 3 years ago

Fixes usage of lists in analysis part of FlowNet YAML configuration file: prescribing a proper list instead of a string

[ ] :tada: This PR closes #241
[ ] :scroll: I have broken down my PR into the following tasks:
- [ ] Changes in the config_parser
- [ ] Changes in create_ert_setup to render list in ERT config file
[ ] :robot: I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
[ ] :book: I have considered adding a new entry in CHANGELOG.md.
[ ] :books: I have considered updating the documentation.

edubarrosTNO commented 3 years ago

I think we might have to merge https://github.com/equinor/flownet-testdata/pull/28 before this one can pass the CI here

wouterjdb commented 3 years ago

I think we might have to merge equinor/flownet-testdata#28 before this one can pass the CI here

No, you can change the github workflow file to temporarily use your fork/branch of the testdata repo to perform the test before merging.

equinor / flownet