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

[Merged by Bors] - chore(zmod/basic): remove unneeded import#18856

Closed
ericrbg wants to merge 1 commit intomasterfrom
ericrbg/tidy_zmod
Closed

[Merged by Bors] - chore(zmod/basic): remove unneeded import#18856
ericrbg wants to merge 1 commit intomasterfrom
ericrbg/tidy_zmod

Conversation

@ericrbg
Copy link
Copy Markdown
Collaborator

@ericrbg ericrbg commented Apr 23, 2023

This file was importing a weird dependency; I am now minimising this.


Open in Gitpod

Will this sort of change be easy to forward-port? If not, I guess this can wait.

@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 Apr 23, 2023
Copy link
Copy Markdown
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.

Yes, this is extremely trivial to forward-port so I'm happy to merge it.

The out-of-sync-length rule is hard to apply right now as the dashboard is stale due to a dead server

bors merge

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Apr 23, 2023
bors bot pushed a commit that referenced this pull request Apr 23, 2023
This file was importing a weird dependency; I am now minimising this.
@bors
Copy link
Copy Markdown

bors bot commented Apr 23, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(zmod/basic): remove unneeded import [Merged by Bors] - chore(zmod/basic): remove unneeded import Apr 23, 2023
@bors bors bot closed this Apr 23, 2023
@bors bors bot deleted the ericrbg/tidy_zmod branch April 23, 2023 23:06
bors bot pushed a commit that referenced this pull request Apr 24, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 24, 2023
kim-em pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 10, 2023
hrmacbeth pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 10, 2023
hrmacbeth pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 11, 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+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants