Skip to content

[Merged by Bors] - feat: port Order.Heyting.Boundary#844

Closed
rosborn wants to merge 4 commits intomasterfrom
Order.Heyting.Boundary
Closed

[Merged by Bors] - feat: port Order.Heyting.Boundary#844
rosborn wants to merge 4 commits intomasterfrom
Order.Heyting.Boundary

Conversation

@rosborn
Copy link
Copy Markdown
Contributor

@rosborn rosborn commented Dec 3, 2022

mathlib SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025

Porting notes:

  1. Needed to fix the proof of boundary_le_boundary_sup_sup_boundary_inf_left as simp produced a different term than in mathlib3
  2. The scoped notation doesn't automatically introduce the notation into the current scope.
    3. The lint error seems like a bug. Can anyone confirm?

@rosborn rosborn added awaiting-review mathlib-port This is a port of a theory file from mathlib. labels Dec 3, 2022
@rosborn
Copy link
Copy Markdown
Contributor Author

rosborn commented Dec 4, 2022

Should I rename hnot to hNot in this PR?

@ChrisHughes24
Copy link
Copy Markdown
Member

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 5, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 6, 2022

bors merge

bors bot pushed a commit that referenced this pull request Dec 6, 2022
mathlib SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025

Porting notes:
1. Needed to fix the proof of `boundary_le_boundary_sup_sup_boundary_inf_left` as simp produced a different term than in mathlib3
2. The `scoped` notation doesn't automatically introduce the notation into the current scope.
~~3. The lint error seems like a bug. Can anyone confirm?~~

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Dec 6, 2022
mathlib SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025

Porting notes:
1. Needed to fix the proof of `boundary_le_boundary_sup_sup_boundary_inf_left` as simp produced a different term than in mathlib3
2. The `scoped` notation doesn't automatically introduce the notation into the current scope.
~~3. The lint error seems like a bug. Can anyone confirm?~~

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

Build failed (retrying...):

  • Build

bors bot pushed a commit that referenced this pull request Dec 6, 2022
mathlib SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025

Porting notes:
1. Needed to fix the proof of `boundary_le_boundary_sup_sup_boundary_inf_left` as simp produced a different term than in mathlib3
2. The `scoped` notation doesn't automatically introduce the notation into the current scope.
~~3. The lint error seems like a bug. Can anyone confirm?~~

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Order.Heyting.Boundary [Merged by Bors] - feat: port Order.Heyting.Boundary Dec 6, 2022
@bors bors bot closed this Dec 6, 2022
@bors bors bot deleted the Order.Heyting.Boundary branch December 6, 2022 02:51
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 7, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.divisibility.units`: leanprover-community/mathlib4#848
* `algebra.group.type_tags`: leanprover-community/mathlib4#832
* `algebra.group_with_zero.divisibility`: leanprover-community/mathlib4#870
* `algebra.hom.equiv.basic`: leanprover-community/mathlib4#835
* `algebra.order.group.defs`: leanprover-community/mathlib4#869
* `algebra.order.monoid.basic`: leanprover-community/mathlib4#872
* `algebra.order.monoid.cancel.basic`: leanprover-community/mathlib4#883
* `algebra.order.monoid.with_zero.defs`: leanprover-community/mathlib4#851
* `algebra.order.monoid.with_zero.basic`: leanprover-community/mathlib4#851
* `algebra.ring.divisibility`: leanprover-community/mathlib4#864
* `data.list.defs`: leanprover-community/mathlib4#803
* `data.sigma.order`: leanprover-community/mathlib4#887
* `group_theory.group_action.defs`: leanprover-community/mathlib4#854
* `order.heyting.boundary`: leanprover-community/mathlib4#844
* `order.hom.basic`: leanprover-community/mathlib4#804
* `order.symm_diff`: leanprover-community/mathlib4#842



Co-authored-by: Johan Commelin <johan@commelin.net>
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.

4 participants