defopt
defopt copied to clipboard
anntzer
Move defopt to package
As the module grows, I think this is not elegant to put separate modules starting with _, but it should be combined in a package.
Actually, I see that napoleon upstream just merged https://github.com/sphinx-contrib/napoleon/pull/36, fixing the 3.6 issue. Let's try to make them release that first, in which case we can go back to having napoleon as a dependency (which is my preferred approach).
Ok, however still it would be much neater not to create an independent module _defopt_version, but collect it into a package.
That part I'll probably completely get rid of and replace by https://github.com/pypa/setuptools_scm#retrieving-package-version-at-runtime.
That part I'll probably completely get rid of and replace by https://github.com/pypa/setuptools_scm#retrieving-package-version-at-runtime.
Good idea!
Done. I think the best thing you could do now is to prod the sphinxcontrib.napoleon maintainer for a new release :-)
Let's just keep things as they are for now. Things work fine in practice (even though the dependency on sphinxcontrib.napoleon is somewhat inelegant), let's not overengineer them; we can always reconsider if there are actual breakages.