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

ContainsOnly #159

Merged
merged 2 commits into from Feb 22, 2015
Merged

ContainsOnly #159

merged 2 commits into from Feb 22, 2015

Conversation

philtay
Copy link
Contributor

@philtay philtay commented Feb 22, 2015

@sloria sloria merged commit ce14427 into marshmallow-code:dev Feb 22, 2015
@sloria
Copy link
Member

sloria commented Feb 22, 2015

Thanks for this! I just made a couple small changes when I merged:

  • Break up tests into smaller units
  • Add a comment explaining why list.index is used.

@sloria sloria mentioned this pull request Feb 22, 2015
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.

None yet

2 participants