Add rich.__version__

[wookayin]

rich should have __version__ property.

AttributeError: module 'rich' has no attribute '__version__'

This is a common python convention to tell package version easily.

[willmcgugan]

I use poetry to do packaging, which recommends pkg_resources to get the version from meta data. They initially recommended dropping __version__ entirely. Fortunately there some to be some movement on that front, and we may get __version__ back.

[wookayin]

Thanks for the feedback. But why is it closed (it's still unimplemented)? Even though poetry is used, we could add this property with a workaround.

[willmcgugan]

Keeping the version in two places is problematic. But if poetry hasn't got a solution when I release a 1.0 version, that's what I might have to do. In the meantime, the lack of a __version__ isn't going to break anything that I know of, and pkg_resources.get_distribution("rich").version can be used to get the version.

[thebigmunch]

Just a note. For Python 3.8+, importlib.metadata should probably be used instead. And the importlib_metadata backaport exists for other Python 3 versions. It was created to replace pkgresources metadata functionality.

Here's how I use it to deduplicate my package metadata. I use poetry, too, and I couldn't wait any longer. I already had that module and was manually updating it.

[willmcgugan]

I may end up doing something similar...

[LewisGaul]

Thanks for the feedback. But why is it closed (it's still unimplemented)? Even though poetry is used, we could add this property with a workaround.

Still seems to be missing __version__, if the intention was for it to be added by v1.0 then perhaps this issue should have been left open...

[willmcgugan]

Don't know where you got that idea from.

[LewisGaul]

I didn't mean to sound adversarial! It was your message above that gave me this impression:

Keeping the version in two places is problematic. But if poetry hasn't got a solution when I release a 1.0 version, that's what I might have to do.

Personally I agree with the OP that it's pretty standard to provide this, but it's up to you of course.

[willmcgugan]

It was a standard of sorts, but now importlib is recommended, and __version__ is a convenience.

Unless poetry has an answer to this that doesn't require a backport of importlib, I'm afraid there is no motivation to change it.