Copy link Copy Markdown

Member

No description provided.

…m.constants`

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, I had seen some places where this was missed. Is there a way to easily test this for all classes (test should probably assume that self.type needs an enum reference)

Copy link Copy Markdown

Member Author

Added a simple regex testcase and actually found another place to apply the change. I excluded passport stuff - we don't have enums for that and I don't like touching that stuff if not really necessary 😬

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The general test case looks good and I'm fine with excluding the passport directory!

kenjitagawa pushed a commit to kenjitagawa/python-telegram-bot that referenced this pull request

Jan 23, 2024