Skip to content
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] - chore: Deprecate pow monotonicity lemmas #9235

Closed
wants to merge 3 commits into from

Conversation

YaelDillies
Copy link
Collaborator

@YaelDillies YaelDillies commented Dec 23, 2023

Add deprecated aliases for all the lemmas removed in #9095 and fix a few renames that were botched.


See https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Renaming.20monotonicity.20of.20powers.20lemmas

Open in Gitpod

Add deprecated aliases for all the lemmas removed in #9095
@YaelDillies YaelDillies added awaiting-review The author would like community review of the PR t-algebra Algebra (groups, rings, fields etc) t-order Order theory labels Dec 23, 2023
Comment on lines -178 to +183
theorem Monotone.pow_right {f : β → M} (hf : Monotone f) : ∀ n : ℕ, Monotone fun a => f a ^ n
theorem Monotone.pow_const {f : β → M} (hf : Monotone f) : ∀ n : ℕ, Monotone fun a => f a ^ n
| 0 => by simpa using monotone_const
| n + 1 => by
simp_rw [pow_succ]
exact hf.mul' (Monotone.pow_right hf _)
#align monotone.pow_right Monotone.pow_right
exact hf.mul' (Monotone.pow_const hf _)
#align monotone.pow_right Monotone.pow_const
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These don't look like deprecations to me; can you split them to their own PR?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those lemma renames were announced in #9095. I just messed up when doing so.

@urkud
Copy link
Member

urkud commented Dec 24, 2023

LGTM but I don't know how to verify that all aliases are there.

@YaelDillies
Copy link
Collaborator Author

You could check the list from the description of #9095 one by one. I copy-pasted it to get the aliases.

@urkud
Copy link
Member

urkud commented Dec 25, 2023

Let's get it merged
bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Dec 25, 2023
mathlib-bors bot pushed a commit that referenced this pull request Dec 25, 2023
Add deprecated aliases for all the lemmas removed in #9095 and fix a few renames that were botched.
@mathlib-bors
Copy link

mathlib-bors bot commented Dec 25, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: Deprecate pow monotonicity lemmas [Merged by Bors] - chore: Deprecate pow monotonicity lemmas Dec 25, 2023
@mathlib-bors mathlib-bors bot closed this Dec 25, 2023
@mathlib-bors mathlib-bors bot deleted the deprecate_pow_mono branch December 25, 2023 17:27
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-algebra Algebra (groups, rings, fields etc) t-order Order theory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants