Skip to content
This repository has been archived by the owner on Jun 15, 2020. It is now read-only.

simple fix for duplicates being removed in smp #74

Merged
merged 1 commit into from
Jan 19, 2017
Merged

simple fix for duplicates being removed in smp #74

merged 1 commit into from
Jan 19, 2017

Conversation

JoeSimmonds
Copy link
Contributor

@JoeSimmonds JoeSimmonds commented Dec 21, 2016 •
edited
Loading

fix for #73
Let me know if this isn't okay and I can sort things. As far as I can tell it hasn't broken any other tests.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 82.075% when pulling 41b026f on JoeSimmonds:master into 0fe0a49 on jto:master.

OlivierBlanvillain
OlivierBlanvillain approved these changes Dec 21, 2016
View reviewed changes
Copy link
Collaborator

@OlivierBlanvillain OlivierBlanvillain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jto jto merged commit 0dddd52 into jto:master Jan 19, 2017
@jto
Copy link
Owner

jto commented Jan 19, 2017

Thanks a lot :)

@JoeSimmonds
Copy link
Contributor Author

No problem, thanks for providing an excellent library.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@OlivierBlanvillain OlivierBlanvillain OlivierBlanvillain approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JoeSimmonds @coveralls @jto @OlivierBlanvillain