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

feat: default serialization for custom objects #101

Merged
merged 8 commits into from
Jan 12, 2020
Merged

Conversation

iamogbz
Copy link
Collaborator

@iamogbz iamogbz commented Jan 12, 2020

Description

Adds a default serialization for custom objects

Related Issues

N/A

Checklist

  • This PR has sufficient test coverage.
  • I have updated the CHANGELOG.md.

Additional Comments

Makes adoption easier

@codecov
Copy link

codecov bot commented Jan 12, 2020

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #101   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          16     16           
  Lines         702    704    +2     
=====================================
+ Hits          702    704    +2

@iamogbz iamogbz requested a review from noahnu January 12, 2020 15:33
@iamogbz iamogbz added feature request New feature or request serializer Syrupy serializer question labels Jan 12, 2020
@iamogbz iamogbz requested a review from noahnu January 12, 2020 16:57
@iamogbz iamogbz requested a review from noahnu January 12, 2020 17:44
@iamogbz iamogbz requested a review from noahnu January 12, 2020 18:13
@iamogbz iamogbz merged commit 6e99629 into master Jan 12, 2020
@iamogbz iamogbz deleted the default-repr branch January 12, 2020 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request released serializer Syrupy serializer question
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants