Skip to content

Conversation

@sjakobi
Copy link
Member

@sjakobi sjakobi commented Oct 6, 2025

As pointed out by @treeowl in #517 (comment).

@treeowl
Copy link
Collaborator

treeowl commented Oct 6, 2025

Redundant tests don't all need to go; they can be good intuition checks. But the ones using ==> with not-particularly-common antecedents are particularly likely to be expensive, as I understand it, because many potential test cases may be discarded. That said, if they're keeping us from running deep and broad enough tests, we can get rid of them all.

@sjakobi sjakobi merged commit a404490 into master Oct 6, 2025
9 checks passed
@sjakobi sjakobi deleted the sjakobi/isSubmapOf branch October 6, 2025 12:58
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.

2 participants