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.

Sign up for GitHub

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

Jump to bottom

v2.2.1 #20

Merged
merged 16 commits into from
Mar 17, 2017
Merged

v2.2.1 #20

merged 16 commits into from
Mar 17, 2017

Conversation

bolorundurowb
Copy link
Owner

PR Type e.g fix, feature, chore etc:

Release

What does this PR do:

Adds a NOTE property and performance improvements

Notes (if any):

Bolorunduro Winner-Timothy Bamidele added 14 commits March 17, 2017 12:25
feature: adds support for serializing REV properties
448d5aa
feature: refactors the deserializers to enable more code reuse
372b5a7
feature: refactors the serializers to enable more code reuse
1439f8f
feature: refactors the serializers to enable more code reuse
d9ef830
feature: makes formatted name property return already retrieved data
af249bb
feature: removes empty or null properties from the serialized string
11a10fb
feature: switches serializer to use string builder instead of string …
e5fed33 
…concatenation
feature: change birthday parsing to use in built parsing methods
c7e808a
feature: adds support for the NOTE property
f0f183a
test: updates the test parameters to cover more lines
2694014
test: updates the test parameters to cover more lines
0c020aa
test: updates the tests for the serializer
89cc543
test: updates the tests for the serializer
7d82b9e
test: fix implementation error
0a80634
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 93.914% when pulling 0a80634 on dev into 492035a on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 93.979% when pulling 0348d89 on dev into 492035a on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.3%) to 95.55% when pulling 690a031 on dev into 492035a on master.

@bolorundurowb bolorundurowb merged commit f068851 into master Mar 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bolorundurowb @coveralls