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

Eliminate many propositional equality imports #2002

Merged
merged 3 commits into from
Jul 6, 2023

Conversation

Taneb
Copy link
Member

@Taneb Taneb commented Jun 22, 2023

Many of the remaining imports of Relation.Binary.PropositionalEquality use either _≗_ or →-to-⟶. Moving _≗_ to Relation.Binary.PropositionalEquality.Core feels like a clear win, except it's defined in a little bit of an indirect way that pulls in some more imports

@Taneb Taneb force-pushed the eliminate-propeq-import branch from 0af4850 to ccd3c81 Compare June 27, 2023 08:16
Copy link
Member

@gallais gallais left a comment

Choose a reason for hiding this comment

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

I think all the alignment changes in Reflection.AST.* suggest we should just do away with lining up the using keywords.

Other than this stylistic remark, everything looks good except for one issue
(perhaps a merge conflict?).

@gallais
Copy link
Member

gallais commented Jun 27, 2023

Also please don't force push the changes if you make any, it'll be a lot easier to
review an additional couple commits than have to go through the whole 200 modules
one by one once again.

Taneb and others added 2 commits June 27, 2023 12:08

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
Co-authored-by: G. Allais <[email protected]>
@Taneb Taneb requested a review from gallais July 6, 2023 06:57
@Taneb Taneb merged commit 177dc9e into agda:master Jul 6, 2023
@Taneb Taneb deleted the eliminate-propeq-import branch July 6, 2023 07:34
plt-amy pushed a commit that referenced this pull request Jul 21, 2023
* Eliminate many propositional equality imports

* Fix merge conflict in Data.Bool.Properties

Co-authored-by: G. Allais <[email protected]>

---------

Co-authored-by: G. Allais <[email protected]>
@andreasabel andreasabel mentioned this pull request Aug 27, 2023
MatthewDaggitt pushed a commit that referenced this pull request Oct 13, 2023
* Eliminate many propositional equality imports

* Fix merge conflict in Data.Bool.Properties

Co-authored-by: G. Allais <[email protected]>

---------

Co-authored-by: G. Allais <[email protected]>
andreasabel pushed a commit that referenced this pull request Jul 10, 2024
* Eliminate many propositional equality imports

* Fix merge conflict in Data.Bool.Properties

Co-authored-by: G. Allais <[email protected]>

---------

Co-authored-by: G. Allais <[email protected]>
@andreasabel andreasabel mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants