search

stac-utils / pystac

Python library for working with any SpatioTemporal Asset Catalog (STAC)
https://pystac.readthedocs.io
Other
357 stars 119 forks source link

str/repr mixup in StringEnum? #1369

Open soxofaan opened 3 months ago

soxofaan commented 3 months ago

https://github.com/stac-utils/pystac/blob/630116c7c31639818b1342c9d2f4231a14dca4f0/pystac/utils.py#L78-L83

Is it intentional to return str(self.value) instead of repr(self.value) here?

jsignell commented 3 months ago

Yeah that seems fine to switch it to repr not sure if it would ever make a difference since by definition the value will always be a string, but I don't see a downside so please open a PR if you feel it's an improvement.