-
Notifications
You must be signed in to change notification settings - Fork 172
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
Replace setoptconf #442
Comments
Oh wow -- a blast from the past. :) If you send over a PR with the fixes you made, I'd be happy to merge them and publish an update to PyPI to hold you over until you're able to completely replace it. |
@jayclassless thanks I made a PR jayclassless/setoptconf#3 |
Merged and published as |
@jayclassless nice, can you also upload a wheel so nobody will be impacted by future setuptools changes? |
Yup, good call. Uploaded. |
Shouldn't we use pydentic? |
Correct me if I'm wrong, but pydantic will not do the command line argument parsing as well? It's not simply about reading the |
Effectively, I completely missed this part... but after a small search I see that there is a project that does that: |
That looks pretty neat, it's worth trying. One concern though - just looking at the README, it gives the example of |
+1 This is an important point, this should be solved :-) |
Oh it might have been - https://github.com/ayasyrev/argparse_pydantic/blob/main/examples/example_05_pydocs.py#L17 |
setoptconf is used for loading configuration values but is an old, stale library; #438 shows that it needs to be updated, or an alternative found.\
(for now a temporary solution is in place)
This issue is here as a placeholder for work done to properly replace it with another library (or alternatively take on maintaining setoptconf completely but... I really don't want to do that).
(ping @jayclassless )
The text was updated successfully, but these errors were encountered: