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

[Merged by Bors] - Feat: Port/Data.Seq.Seq #3187

Closed
wants to merge 60 commits into from
Closed

Conversation

arienmalec
Copy link
Collaborator

@arienmalec arienmalec commented Mar 30, 2023

Port of Data.Seq.Seq following Mathlib3 refactor to avoid Seq overload.

Down to two issues:

  • bind_ret goes like Mathlib3 right up to the last line but doesn't close (I've tried the full Mathlib3 simp invocation to no avail.
  • bind_assoc has issues with single match lines gumming up the works, which should dsimp away but don't

Open in Gitpod

Mathbin -> Mathlib
fix certain import statements
move "by" to end of line
add import to Mathlib.lean
@arienmalec arienmalec added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. labels Mar 30, 2023
@arienmalec arienmalec added help-wanted The author needs attention to resolve issues and removed WIP Work in progress labels Apr 2, 2023
@arienmalec

This comment was marked as duplicate.

@Komyyy Komyyy added awaiting-CI and removed help-wanted The author needs attention to resolve issues labels Apr 6, 2023
@Komyyy Komyyy added the awaiting-review The author would like community review of the PR label Apr 6, 2023
@ChrisHughes24
Copy link
Member

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Apr 10, 2023
bors bot pushed a commit that referenced this pull request Apr 10, 2023
Port of `Data.Seq.Seq` following Mathlib3 refactor to avoid `Seq` overload.

Down to two issues:

- [x] `bind_ret` goes like Mathlib3 right up to the last line but doesn't close (I've tried the full Mathlib3 `simp` invocation to no avail.
- [x] `bind_assoc` has issues with single `match` lines gumming up the works, which should `dsimp` away but don't




Co-authored-by: Komyyy <pol_tta@outlook.jp>
@bors
Copy link

bors bot commented Apr 10, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Feat: Port/Data.Seq.Seq [Merged by Bors] - Feat: Port/Data.Seq.Seq Apr 10, 2023
@bors bors bot closed this Apr 10, 2023
@bors bors bot deleted the port/Data.Seq.Seq branch April 10, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mathlib-port This is a port of a theory file from mathlib. ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants