Skip to content

Implement wrappers for condition classes#709

Open
hadley wants to merge 1 commit into
mainfrom
condition-classes
Open

Implement wrappers for condition classes#709
hadley wants to merge 1 commit into
mainfrom
condition-classes

Conversation

@hadley

@hadley hadley commented Jun 19, 2026

Copy link
Copy Markdown
Member

Fixes #586

@sebffischer

Copy link
Copy Markdown

I like it. Although this is rather minimal and does not provide a lot of functionality itself, I think condition objects are fundamental to the language, so I personally think they do deserve base classes.

I also like what you said in the issue:

OTOH maybe this could be a place where we could provide more details for people who do want to extend condition classes for their own purposes?

For completeness sake, condition_message should maybe also be included here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: add class_error, class_condition, class_warning base classes

2 participants