Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - chore(ring_theory/kaehler): cleanup instances#19234

Closed
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/golf-kaehler
Closed

[Merged by Bors] - chore(ring_theory/kaehler): cleanup instances#19234
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/golf-kaehler

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

The previous module instance has the wrong defeqs, and was more work to construct.
The new one remains propositionally equal to the old one.


Open in Gitpod

The previous module instance has the wrong defeqs, and was more work to construct.
The new one remains propositionally equal to the old one.
@eric-wieser eric-wieser added the awaiting-review The author would like community review of the PR label Jul 12, 2023
@github-actions github-actions bot added the modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. label Jul 12, 2023
@eric-wieser eric-wieser added the t-algebra Algebra (groups, rings, fields etc) label Jul 12, 2023
eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request Jul 13, 2023
eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request Jul 13, 2023
eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request Jul 14, 2023
eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request Jul 14, 2023
@eric-wieser eric-wieser added the not-too-late This PR was ready at the point mathlib3 was frozen: we will try to merge it and port it to mathlib4 label Jul 15, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jul 16, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jul 16, 2023
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@riccardobrasca riccardobrasca self-assigned this Jul 19, 2023
@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Jul 24, 2023
bors bot pushed a commit that referenced this pull request Jul 24, 2023
The previous module instance has the wrong defeqs, and was more work to construct.
The new one remains propositionally equal to the old one.
@bors
Copy link
Copy Markdown

bors bot commented Jul 24, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title chore(ring_theory/kaehler): cleanup instances [Merged by Bors] - chore(ring_theory/kaehler): cleanup instances Jul 24, 2023
@bors bors bot closed this Jul 24, 2023
@bors bors bot deleted the eric-wieser/golf-kaehler branch July 24, 2023 08:57
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jul 24, 2023
Unfortunately I was not able to forward port the addition of a partially-explicit type to `kaehler_differential.End_equiv_derivation'`, as Lean 4 no longer allows metavariables in instance arguments.
fgdorais pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jul 25, 2023
Unfortunately I was not able to forward port the addition of a partially-explicit type to `kaehler_differential.End_equiv_derivation'`, as Lean 4 no longer allows metavariables in instance arguments.
kim-em added a commit to leanprover-community/mathlib4 that referenced this pull request Aug 14, 2023
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
kim-em pushed a commit to leanprover-community/mathlib4 that referenced this pull request Aug 14, 2023
Unfortunately I was not able to forward port the addition of a partially-explicit type to `kaehler_differential.End_equiv_derivation'`, as Lean 4 no longer allows metavariables in instance arguments.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. not-too-late This PR was ready at the point mathlib3 was frozen: we will try to merge it and port it to mathlib4 ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) t-algebra Algebra (groups, rings, fields etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants