Skip to content

Don't derive Show for Unique#1127

Merged
parsonsmatt merged 2 commits intomasterfrom
988-no-show-unique
Sep 18, 2020
Merged

Don't derive Show for Unique#1127
parsonsmatt merged 2 commits intomasterfrom
988-no-show-unique

Conversation

@parsonsmatt
Copy link
Collaborator

@parsonsmatt parsonsmatt commented Sep 18, 2020

Fixes #988

Before submitting your PR, check that you've:

After submitting your PR:

  • Update the Changelog.md file with a link to your PR
  • Bumped the version number if there isn't an (unreleased) on the Changelog
  • Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)

@parsonsmatt parsonsmatt added this to the 2.11 milestone Sep 18, 2020
@parsonsmatt parsonsmatt merged commit 0c3d32d into master Sep 18, 2020
@parsonsmatt parsonsmatt deleted the 988-no-show-unique branch April 20, 2021 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requirement of a Show instance for Unique is not good

1 participant