Skip to content
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

Move proofs of reducibility to theories/LogicalRelation/Introductions. #66

Merged
merged 4 commits into from
Jan 31, 2025

Conversation

ppedrot
Copy link
Contributor

@ppedrot ppedrot commented Jan 28, 2025

This parallels the organization of the Validity folder.

This parallels the organization of the Validity folder.
This ensures a clean delineation of dependencies in LogicalRelation.Introductions.
This effectively moves the reducibility-only parts of Poly and Pi from
Validity to LogicalRelation.
@MevenBertrand
Copy link
Member

Btw, I don't know if you used it, but the depgraph make recipe is in my experience quite useful to detect suspicious dependencies. I did quite a bit of cleanup on the algo side using it, although I did not dare touching the logrel one.

@ppedrot
Copy link
Contributor Author

ppedrot commented Jan 29, 2025

Indeed, I realized there was an issue thanks to depgraph.

@kyoDralliam kyoDralliam merged commit 43f28a3 into CoqHott:coq-8.20 Jan 31, 2025
1 check passed
@ppedrot ppedrot deleted the move-logrel-intro branch January 31, 2025 09:23
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