FS#67854 - [python-toml] install type hints

Attached to Project: Community Packages
Opened by Filipe Laíns (FFY00) - Wednesday, 09 September 2020, 00:10 GMT
Last edited by David Runge (dvzrv) - Sunday, 13 September 2020, 20:19 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To David Runge (dvzrv)
Daniel M. Capella (polyzen)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The project provides a `toml.pyi` file with type hints but it is not installed. Would it be possible to install it?
This task depends upon

Closed by  David Runge (dvzrv)
Sunday, 13 September 2020, 20:19 GMT
Reason for closing:  Upstream
Additional comments about closing:  This ultimately is still an upstream issue and therefore should be dealt with there. Eventually changes there will make it to our packages anyways.

However, as you have already supplied a patch to upstream I have added the file as well.

python-toml 0.10.1-2
Comment by Daniel M. Capella (polyzen) - Sunday, 13 September 2020, 05:35 GMT Comment by David Runge (dvzrv) - Sunday, 13 September 2020, 12:42 GMT
@FFY00: Thanks for the report!

As much as I like the type hints to be included I think this ultimately to be an upstream issue.
Both tickets linked to by @polyzen are relevant here and once they are resolved the file should a) be correct and b) be installed by setuptools.
Comment by Filipe Laíns (FFY00) - Sunday, 13 September 2020, 14:31 GMT
I can't request to re-open, just re-open, so I am sorry. Feel free to close if you still don't agree.

As referred in issue #304, the type hints are incomplete, not incorrect. Part of the API still doesn't have type hints, but this shouldn't block us from installing the type hints.

I don't think the upstream *needs* to make sure the type hints are installed via setuptools for us to provide them in the package, we could still install the file ourselves. I understand if you disagree.
Comment by Filipe Laíns (FFY00) - Sunday, 13 September 2020, 16:27 GMT
I have submitted a patch upstream, it could be backported if you choose to install the type stubs, but a simple install -Dm 644 would also do.

https://github.com/uiri/toml/pull/318

Loading...