Skip to content

[Merged by Bors] - chore: make the 'p' argument to Nat.find implicit#730

Closed
Ruben-VandeVelde wants to merge 1 commit intomasterfrom
Nat.find-implicit
Closed

[Merged by Bors] - chore: make the 'p' argument to Nat.find implicit#730
Ruben-VandeVelde wants to merge 1 commit intomasterfrom
Nat.find-implicit

Conversation

@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

This matches lean3.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 25, 2022

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Nov 25, 2022
bors bot pushed a commit that referenced this pull request Nov 25, 2022
@bors
Copy link
Copy Markdown

bors bot commented Nov 25, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore: make the 'p' argument to Nat.find implicit [Merged by Bors] - chore: make the 'p' argument to Nat.find implicit Nov 25, 2022
@bors bors bot closed this Nov 25, 2022
@bors bors bot deleted the Nat.find-implicit branch November 25, 2022 21:50
@digama0
Copy link
Copy Markdown
Member

digama0 commented Nov 26, 2022

This was a deliberate change. Proof arguments in definitions should not make other arguments implicit. (It should be backported if the difference is causing problems.)

rosborn added a commit that referenced this pull request Nov 27, 2022
* master:
  feat: port Data.Countable.Defs (#736)
  chore: bump to nightly-2022-11-26 (#747)
  feat(Algebra/Ring/Units): port file (#746)
  style(Algebra/Order/Monoid/Lemmas): Update to current naming convention (#743)
  feat: stubs in ad-hoc ported files for linarith algebra requirements (#733)
  feat: port algebra.group.semiconj (#717)
  chore: make argument to mul_inv_cancel implicit (#737)
  feat(Algebra/Ring/InjSurj): port file (#734)
  chore: bump to nightly-2022-11-25 (#731)
  feat: port order.minmax (#728)
  chore: make the 'p' argument to Nat.find implicit (#730)
  feat: port Control.Basic (#588)
  feat: port data.finite.defs (#698)
  feat: port:  Data.DList.Basic (#616)
  chore: reduce imports in Algebra.Group.Defs (#727)
  chore(Algebra/Order/Hom/Basic): remove outParam (#692)
bors bot pushed a commit that referenced this pull request Nov 30, 2022
This is based on a *future* version of mathlib3, once leanprover-community/mathlib3#17763 and leanprover-community/mathlib3#17759 have landed.

- [x] depends on #727
- [x] depends on #730

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants