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(algebra/algebra): implement subalgebra.under in terms of restrict_scalars #10080

Closed
wants to merge 2 commits into from

Conversation

eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Oct 31, 2021

We should probably remove subalgebra.under entirely, but that's likely a lot more work.


Open in Gitpod

@eric-wieser eric-wieser added the awaiting-review The author would like community review of the PR label Oct 31, 2021
@fpvandoorn
Copy link
Member

We should maybe add a note that we want to remove this.

LGTM

bors d+

@bors
Copy link

bors bot commented Nov 1, 2021

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

@github-actions github-actions bot 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 Nov 1, 2021
@eric-wieser
Copy link
Member Author

bors merge

We have an open PR at #10081, so I don't think a note is that important :)

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Nov 1, 2021
bors bot pushed a commit that referenced this pull request Nov 1, 2021
…ct_scalars (#10080)

We should probably remove `subalgebra.under` entirely, but that's likely a lot more work.
@bors
Copy link

bors bot commented Nov 1, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(algebra/algebra): implement subalgebra.under in terms of restrict_scalars [Merged by Bors] - chore(algebra/algebra): implement subalgebra.under in terms of restrict_scalars Nov 1, 2021
@bors bors bot closed this Nov 1, 2021
@bors bors bot deleted the eric-wieser/subalgebra.under branch November 1, 2021 12:22
ericrbg pushed a commit that referenced this pull request Nov 9, 2021
…ct_scalars (#10080)

We should probably remove `subalgebra.under` entirely, but that's likely a lot more work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants