Skip to content

Change ⊢⇌ to in optimizations tests #823

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

Jacob-Lockwood
Copy link
Contributor

The documentation aspect of this was addressed in commit 5dba9c9, but the optimizations tests were never changed to reflect it.

@kaikalii
Copy link
Member

It's like that on purpose. The first rev to last optimization is also being tested.

@Jacob-Lockwood
Copy link
Contributor Author

Maybe it would be better to add a separate test for that, then.

@Omnikar
Copy link
Collaborator

Omnikar commented Aug 15, 2025

It's not a separate test. first reverse rise and friends are their own optimizations.

@Jacob-Lockwood
Copy link
Contributor Author

Interesting. I wonder if that's a good thing? Still, I'll close the PR.

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.

3 participants