Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

feat(topology/order): Add closure.mono#19224

Closed
mans0954 wants to merge 2 commits intomasterfrom
mans0954/Closure.mono
Closed

feat(topology/order): Add closure.mono#19224
mans0954 wants to merge 2 commits intomasterfrom
mans0954/Closure.mono

Conversation

@mans0954
Copy link
Copy Markdown
Collaborator

Adds closure.mono which asserts that if t₁ ≤ t₂ for topologies t₁, t₂, then the closure of a set in t₁ will be a subset of the closure of the set in t₂. Analogous to is_open.mono and is_closed.mono.


See discussion

Open in Gitpod

@github-actions github-actions bot added the modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. label Jun 30, 2023
@mans0954
Copy link
Copy Markdown
Collaborator Author

I think I'm now meant to create an empty "holding" PR in mathlib4 - but it doesn't seem to be possible to create empty PRs?

@mans0954 mans0954 added the awaiting-review The author would like community review of the PR label Jun 30, 2023
@j-loreaux
Copy link
Copy Markdown
Collaborator

You should just close this and PR to mathlib 4 instead. It will be less work for you.

@mans0954
Copy link
Copy Markdown
Collaborator Author

You should just close this and PR to mathlib 4 instead. It will be less work for you.

For files that exist in both mathlib and mathlib4 I thought it was mandatory to get the PR merged in mathlib and then forward port to mathlib4?

@mans0954
Copy link
Copy Markdown
Collaborator Author

Closing in favour of leanprover-community/mathlib4#5631

@mans0954 mans0954 closed this Jun 30, 2023
@mans0954 mans0954 deleted the mans0954/Closure.mono branch July 7, 2023 07:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

awaiting-review The author would like community review of the PR modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants