leanprover-community / leanprover-community/mathlib4
clean up completely normal results to use `SeparatedNhds`
Open
@StevenClontz is already working on this.
Since Jun 27, 2024.
t-topology
- Dominant language
- Lean
- Stars
- 4.2k
- Forks
- 1.7k
- PR merge metrics
- No merged PRs in 30d
Description
Sticking a pin in a chore I will follow up #13817 with.
@JADekker:
The changes look good to me, but I have the feeling that most proofs that you touched seem to become longer after this change (now requiring some form of
separatedNhds_iff_disjoint), which makes me wonder if these proofs can be simplified if we also useSeparatedNhdsin another place, or if it may be more natural to formulate both definitions in terms ofDisjointinstead. Any thoughts?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.