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 ] concat [ xs ] ≡ xs #2530

Merged
merged 5 commits into from
Dec 28, 2024
Merged

Conversation

jamesmckinna
Copy link
Contributor

@jamesmckinna jamesmckinna commented Dec 20, 2024

I'm persuaded by the discussion on Zulip (and my own thoughtlessness in responding there) that this is a simple worthwhile addition, with its potential downstream use in #2524 (and elsewhere). Thanks to @ncfavier for clarifying the triangle laws for the List monad!

Possible issue(s):

  • naming (as ever); alternatives concat∘[_]≗id? (or some prefix version?)
  • use of mixfix notation; but here it seems to chime well with [_] as an operator?
  • rename existing concat-[-] to eg. concat∘map[_]≗id? (or some more informative prefix version?)

src/Data/List/Properties.agda Show resolved Hide resolved
@JacquesCarette JacquesCarette added this pull request to the merge queue Dec 28, 2024
Merged via the queue into agda:master with commit cd7963b Dec 28, 2024
2 checks passed
@jamesmckinna jamesmckinna deleted the concat-pure branch December 28, 2024 15:24
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.

4 participants