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

chore(ring_theory/polynomial): remove decidable_eq assumptions#1890

Merged
mergify[bot] merged 2 commits intomasterfrom
hilbert_dec_eq
Jan 18, 2020
Merged

chore(ring_theory/polynomial): remove decidable_eq assumptions#1890
mergify[bot] merged 2 commits intomasterfrom
hilbert_dec_eq

Conversation

@ChrisHughes24
Copy link
Copy Markdown
Member

@ChrisHughes24 ChrisHughes24 commented Jan 17, 2020

mv_polynomial is now noncomputable so these arguments are unnecessary

TO CONTRIBUTORS:

Make sure you have:

  • reviewed and applied the coding style: coding, naming
  • reviewed and applied the documentation requirements
  • for tactics:
  • make sure definitions and lemmas are put in the right files
  • make sure definitions and lemmas are not redundant

If this PR is related to a discussion on Zulip, please include a link in the discussion.

For reviewers: code review check list

@jcommelin jcommelin 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 Jan 18, 2020
@mergify mergify bot merged commit d548d92 into master Jan 18, 2020
@mergify mergify bot deleted the hilbert_dec_eq branch January 18, 2020 05:20
butterthebuddha pushed a commit to butterthebuddha/mathlib that referenced this pull request May 15, 2020
…rover-community#1890)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
butterthebuddha pushed a commit to butterthebuddha/mathlib that referenced this pull request May 16, 2020
…rover-community#1890)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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.

2 participants