Skip to content

[Merged by Bors] - chore: rename {LocalHomeomorph,ChartedSpace}.continuous_{to,inv}Fun fields to continuousOn_{to,inv}Fun#8848

Closed
grunweg wants to merge 3 commits intomasterfrom
MR-localhom-rename
Closed

[Merged by Bors] - chore: rename {LocalHomeomorph,ChartedSpace}.continuous_{to,inv}Fun fields to continuousOn_{to,inv}Fun#8848
grunweg wants to merge 3 commits intomasterfrom
MR-localhom-rename

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Dec 6, 2023

They have type ContinuousOn ..., hence should be named accordingly. Suggested by @fpvandoorn in #8736.


None of these fields is ever used in a doc comment. Best reviewed commit-by-commit; the first was created merely by renaming.
I can generate the leaff output for this PR if that's helpful; let me know.

Created using the new rename handler. @digama0 This worked like a charm; thanks for creating this!


Open in Gitpod

@grunweg grunweg added awaiting-review t-differential-geometry Manifolds etc t-topology Topological spaces, uniform spaces, metric spaces, filters labels Dec 6, 2023
@fpvandoorn
Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 7, 2023
mathlib-bors bot pushed a commit that referenced this pull request Dec 7, 2023
…ields to continuousOn_{to,inv}Fun (#8848)

They have type ContinuousOn ..., hence should be named accordingly. Suggested by @fpvandoorn in #8736.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Dec 7, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: rename {LocalHomeomorph,ChartedSpace}.continuous_{to,inv}Fun fields to continuousOn_{to,inv}Fun [Merged by Bors] - chore: rename {LocalHomeomorph,ChartedSpace}.continuous_{to,inv}Fun fields to continuousOn_{to,inv}Fun Dec 7, 2023
@mathlib-bors mathlib-bors bot closed this Dec 7, 2023
@mathlib-bors mathlib-bors bot deleted the MR-localhom-rename branch December 7, 2023 01:15
awueth pushed a commit that referenced this pull request Dec 19, 2023
…ields to continuousOn_{to,inv}Fun (#8848)

They have type ContinuousOn ..., hence should be named accordingly. Suggested by @fpvandoorn in #8736.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-differential-geometry Manifolds etc t-topology Topological spaces, uniform spaces, metric spaces, filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants