Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support for nested field types #3

Merged
merged 2 commits into from
Dec 4, 2019
Merged

Support for nested field types #3

merged 2 commits into from
Dec 4, 2019

Conversation

devinaconley
Copy link
Owner

  • allow setting class type for serializable Nested field
  • allow setting class type for nested serializable List
  • will automatically use this class if _type is not specified in dictionary
  • will throw error if conflicting _type is provided
  • bump to 0.0.3

@codecov-io
Copy link

Codecov Report

Merging #3 into develop will decrease coverage by 0.76%.
The diff coverage is 95.08%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop       #3      +/-   ##
===========================================
- Coverage    99.31%   98.54%   -0.77%     
===========================================
  Files            7        7              
  Lines          290      344      +54     
===========================================
+ Hits           288      339      +51     
- Misses           2        5       +3
Impacted Files Coverage Δ
objectfactory/__init__.py 100% <100%> (ø) ⬆️
test/test_fields.py 100% <100%> (ø) ⬆️
objectfactory/serializable.py 100% <100%> (ø) ⬆️
objectfactory/field.py 88.37% <84.21%> (-4.49%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b5fa49...38d8ded. Read the comment docs.

@devinaconley
Copy link
Owner Author

Travis build passed

@devinaconley devinaconley merged commit f4158af into develop Dec 4, 2019
@devinaconley devinaconley deleted the type_0.0.3 branch December 4, 2019 21:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants