Skip to content

[Merged by Bors] - chore: separate NeZero dependency from Nat/Cast/Defs#6955

Closed
kim-em wants to merge 2 commits intomasterfrom
nat_cast_nezero
Closed

[Merged by Bors] - chore: separate NeZero dependency from Nat/Cast/Defs#6955
kim-em wants to merge 2 commits intomasterfrom
nat_cast_nezero

Conversation

@kim-em
Copy link
Copy Markdown
Contributor

@kim-em kim-em commented Sep 5, 2023

I'm trying to remove any extraneous material from the core definitions in the algebraic hierarchy that are used in tactics.


Open in Gitpod

@kim-em kim-em added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. labels Sep 5, 2023
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Sep 5, 2023
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Sep 5, 2023
bors bot pushed a commit that referenced this pull request Sep 5, 2023
I'm trying to remove any extraneous material from the core definitions in the algebraic hierarchy that are used in tactics.



Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
bors bot pushed a commit that referenced this pull request Sep 5, 2023
This continues reducing the import requirements for the basic algebraic hierarchy.

In combination with #6954 #6955 #6956 #6957, this reduces the imports for `Mathlib.Algebra.Field.Defs` from

* [before.pdf](https://github.com/leanprover-community/mathlib4/files/12518547/before.pdf)
* [after.pdf](https://github.com/leanprover-community/mathlib4/files/12518546/after.pdf)




Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Sep 5, 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: separate NeZero dependency from Nat/Cast/Defs [Merged by Bors] - chore: separate NeZero dependency from Nat/Cast/Defs Sep 5, 2023
@bors bors bot closed this Sep 5, 2023
@bors bors bot deleted the nat_cast_nezero branch September 5, 2023 10:31
ebab pushed a commit that referenced this pull request Sep 6, 2023
I'm trying to remove any extraneous material from the core definitions in the algebraic hierarchy that are used in tactics.



Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
ebab pushed a commit that referenced this pull request Sep 6, 2023
This continues reducing the import requirements for the basic algebraic hierarchy.

In combination with #6954 #6955 #6956 #6957, this reduces the imports for `Mathlib.Algebra.Field.Defs` from

* [before.pdf](https://github.com/leanprover-community/mathlib4/files/12518547/before.pdf)
* [after.pdf](https://github.com/leanprover-community/mathlib4/files/12518546/after.pdf)




Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
kodyvajjha pushed a commit that referenced this pull request Sep 22, 2023
I'm trying to remove any extraneous material from the core definitions in the algebraic hierarchy that are used in tactics.



Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
kodyvajjha pushed a commit that referenced this pull request Sep 22, 2023
This continues reducing the import requirements for the basic algebraic hierarchy.

In combination with #6954 #6955 #6956 #6957, this reduces the imports for `Mathlib.Algebra.Field.Defs` from

* [before.pdf](https://github.com/leanprover-community/mathlib4/files/12518547/before.pdf)
* [after.pdf](https://github.com/leanprover-community/mathlib4/files/12518546/after.pdf)




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.

2 participants