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

Add monoidal & enriched categories #665

Merged
merged 4 commits into from
Dec 17, 2021
Merged

Conversation

barrettj12
Copy link
Contributor

@barrettj12 barrettj12 commented Dec 14, 2021

This PR depends on #654, so that will have to be merged first.

Will close #643.

Copy link
Collaborator

@mortberg mortberg left a comment

Choose a reason for hiding this comment

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

Looks mostly good to me, just two small comments/questions on naming. Note that there is now a conflict that needs to be fixed before merging

Cubical/Categories/Monoidal/Base.agda Outdated Show resolved Hide resolved
Cubical/Categories/Monoidal/Base.agda Outdated Show resolved Hide resolved
@mortberg
Copy link
Collaborator

There is still a conflict

@barrettj12
Copy link
Contributor Author

There is still a conflict

Whoops forgot, fixed now.

@mortberg
Copy link
Collaborator

There is still a conflict

Whoops forgot, fixed now.

Great! Merging once CI is done

@mortberg mortberg merged commit ad75bec into agda:master Dec 17, 2021
@barrettj12 barrettj12 deleted the monoidal2 branch December 17, 2021 10:48
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.

Make _⋆_, _∘_ right associative in Category
2 participants