bw2 / ConfigArgParse

A drop-in replacement for argparse that allows options to also be set via config files and/or environment variables.
MIT License
719 stars 121 forks source link

Add a test and remove windows from the CI for now #268

Closed tristanlatr closed 1 year ago

tristanlatr commented 2 years ago

This PR only adds a test. I wasn't sure why some functionalities wasn't working with my TOML config parser, so I decided to write. test with the YAML config parser that mimics my issue.

It passes as expected :) So there is an issue with my code.

Anyway, I figured you'd maybe wanted to have this test anyhow.

tristanlatr commented 2 years ago

The tests seem to fail with ubuntu-latest py3.8 but not with macos :/

Note that the windows tests were broken before.

tristanlatr commented 2 years ago

Hi @bw2, this PR fixes the CI. I hope is can be merged soon.

Thanks

tristanlatr commented 2 years ago

Up!