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

Conversation

@YaelDillies
Copy link
Collaborator

@YaelDillies YaelDillies commented Mar 16, 2023

Move the archimedean and floor_ring instances out of algebra.order.nonneg.field into a new file algebra.order.nonneg.floor.


This is crucial for unmangling imports and defining nnrat.cast in #16554.

Open in Gitpod

@YaelDillies YaelDillies added awaiting-review The author would like community review of the PR t-algebra Algebra (groups, rings, fields etc) t-order Order hierarchy labels Mar 16, 2023
Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

Thanks!

@bors
Copy link

bors bot commented Mar 16, 2023

✌️ YaelDillies can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Mar 16, 2023
@eric-wieser eric-wieser added awaiting-review The author would like community review of the PR modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. and removed delegated The PR author may merge after reviewing final suggestions. labels Mar 16, 2023
@eric-wieser
Copy link
Member

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 Mar 16, 2023
bors bot pushed a commit that referenced this pull request Mar 16, 2023
…8596)

Move the `archimedean` and `floor_ring` instances out of `algebra.order.nonneg.field` into a new file `algebra.order.nonneg.floor`.
@bors
Copy link

bors bot commented Mar 16, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(algebra/order/nonneg/*): Separate floor_ring from field [Merged by Bors] - chore(algebra/order/nonneg/*): Separate floor_ring from field Mar 16, 2023
@bors bors bot closed this Mar 16, 2023
@bors bors bot deleted the split_nonneg_floor branch March 16, 2023 17:03
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 17, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 17, 2023
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. 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) t-order Order hierarchy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants