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 Topology.Basic #1826
Conversation
urkud
commented
Jan 25, 2023
Mathbin -> Mathlib fix certain import statements move "by" to end of line add import to Mathlib.lean
Co-authored-by: Frédéric Dupuis <31101893+dupuisf@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, except for a naming question about lim_under
/limUnder
.
Mathlib/Topology/Basic.lean
Outdated
noncomputable def lim [Nonempty α] (f : Filter β) (g : β → α) : α := | ||
/-- If `f` is a filter in `β` and `g : β → α` is a function, then `lim_under f g` is a limit of `g` | ||
at `f`, if it exists. -/ | ||
noncomputable def lim_under [Nonempty α] (f : Filter β) (g : β → α) : α := |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the naming convention would suggest camelCasing this:
noncomputable def lim_under [Nonempty α] (f : Filter β) (g : β → α) : α := | |
noncomputable def limUnder [Nonempty α] (f : Filter β) (g : β → α) : α := |
bors d+ |
✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
Co-authored-by: Johan Commelin <johan@commelin.net>
Pull request successfully merged into master. Build succeeded:
|
These files have been primarily modified by backports and need little modification: * `topology.basic`: #1826 - modified with a porting note, which can now be removed * `data.real.cau_seq_completion`: #1469 - not a backport, but forgot to update the SHA * `order.filter.n_ary.basic`: #1967 - this PR forgot to update the SHA * `ring_theory.valuation.basic`: The change is a small golf that is now included in this PR Co-authored-by: Eric Wieser <wieser.eric@gmail.com>