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

chore(algebra): swap dependency order of algebra.group_power.ring and algebra.ne_zero #17175

Closed
wants to merge 1 commit into from

Conversation

semorrison
Copy link
Collaborator


Open in Gitpod

@semorrison semorrison added awaiting-review The author would like community review of the PR awaiting-CI The author would like to see what CI has to say before doing more work. labels Oct 26, 2022
@semorrison
Copy link
Collaborator Author

Superceded by #17177.

@semorrison semorrison closed this Oct 26, 2022
bors bot pushed a commit that referenced this pull request Nov 1, 2022
This removes basically everything from `algebra.ne_zero` (since it doesn't require any imports for the definition), and inverts the dependencies on all the theorems in the file, putting them wherever they can fit earliest.

Supercedes #17175 and (I think) #17176.
bors bot pushed a commit that referenced this pull request Nov 1, 2022
This removes basically everything from `algebra.ne_zero` (since it doesn't require any imports for the definition), and inverts the dependencies on all the theorems in the file, putting them wherever they can fit earliest.

Supercedes #17175 and (I think) #17176.
bors bot pushed a commit that referenced this pull request Nov 4, 2022
This removes basically everything from `algebra.ne_zero` (since it doesn't require any imports for the definition), and inverts the dependencies on all the theorems in the file, putting them wherever they can fit earliest.

Supercedes #17175 and (I think) #17176.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
bors bot pushed a commit that referenced this pull request Nov 4, 2022
This removes basically everything from `algebra.ne_zero` (since it doesn't require any imports for the definition), and inverts the dependencies on all the theorems in the file, putting them wherever they can fit earliest.

Supercedes #17175 and (I think) #17176.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
bors bot pushed a commit that referenced this pull request Nov 4, 2022
This removes basically everything from `algebra.ne_zero` (since it doesn't require any imports for the definition), and inverts the dependencies on all the theorems in the file, putting them wherever they can fit earliest.

Supercedes #17175 and (I think) #17176.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@YaelDillies YaelDillies deleted the semorrison/swap_group_power_ring_ne_zero branch February 20, 2023 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-CI The author would like to see what CI has to say before doing more work. awaiting-review The author would like community review of the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant