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

Low hanging fruit regarding limits #410

Merged
merged 9 commits into from
Jul 19, 2024
Merged

Low hanging fruit regarding limits #410

merged 9 commits into from
Jul 19, 2024

Conversation

TOTBWF
Copy link
Collaborator

@TOTBWF TOTBWF commented Jul 16, 2024

Description

This PR fills in a bunch of low hanging fruit from chapter 2 of Borceux, Vol 1.

Checklist

Before submitting a merge request, please check the items below:

  • I've read the contributing guidelines.
  • The imports of new modules have been sorted with support/sort-imports.hs (or nix run --experimental-features nix-command -f . sort-imports).
  • All new code blocks have "agda" as their language.

If your change affects many files without adding substantial content, and
you don't want your name to appear on those pages (for example, treewide
refactorings or reformattings), start the commit message and PR title with chore:.

@TOTBWF TOTBWF requested review from ncfavier and plt-amy July 16, 2024 23:29
@Lavenza
Copy link
Member

Lavenza commented Jul 17, 2024

Pull request preview

Copy link
Member

@ncfavier ncfavier left a comment

Choose a reason for hiding this comment

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

\id

src/Cat/Diagram/Equaliser.lagda.md Outdated Show resolved Hide resolved
src/Cat/Diagram/Pullback.lagda.md Outdated Show resolved Hide resolved
src/Cat/Diagram/Pullback.lagda.md Outdated Show resolved Hide resolved
src/Cat/Diagram/Pullback.lagda.md Outdated Show resolved Hide resolved
src/Cat/Diagram/Pullback.lagda.md Outdated Show resolved Hide resolved
@plt-amy plt-amy enabled auto-merge (squash) July 19, 2024 19:56
@plt-amy plt-amy force-pushed the limit-properties branch from a33e01d to 7c0c34d Compare July 19, 2024 19:56
@plt-amy plt-amy merged commit 40733b6 into main Jul 19, 2024
5 checks passed
@plt-amy plt-amy deleted the limit-properties branch July 19, 2024 20:12
TOTBWF added a commit that referenced this pull request Jul 23, 2024
plt-amy pushed a commit that referenced this pull request Jul 25, 2024
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.

4 participants