str/repr mixup in StringEnum?

Open soxofaan opened this issue 1 year ago • 1 comments

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?

Jul 23 '24 13:07 soxofaan

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.

Jul 24 '24 13:07 jsignell