Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Implement defaultProps #79

Merged
merged 10 commits into from
May 3, 2018
Merged

Implement defaultProps #79

merged 10 commits into from
May 3, 2018

Conversation

LPGhatguy
Copy link
Contributor

@LPGhatguy LPGhatguy commented May 1, 2018 •
edited
Loading

Closes #78.

TODO:

  • Tests
  • Docs

@LPGhatguy LPGhatguy self-assigned this May 1, 2018
@coveralls
Copy link

coveralls commented May 1, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.1%) to 89.513% when pulling 449e952 on default-props into 3169897 on master.

@LPGhatguy LPGhatguy changed the title WIP: Implement defaultProps Implement defaultProps May 3, 2018
@LPGhatguy LPGhatguy mentioned this pull request May 3, 2018
Closed
ZoteTheMighty
ZoteTheMighty approved these changes May 3, 2018
View reviewed changes
Copy link
Contributor

@ZoteTheMighty ZoteTheMighty left a comment

Choose a reason for hiding this comment

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

LGTM!

@LPGhatguy LPGhatguy merged commit e42c2a2 into master May 3, 2018
@LPGhatguy LPGhatguy deleted the default-props branch May 3, 2018 19:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ZoteTheMighty ZoteTheMighty ZoteTheMighty approved these changes

Assignees

@LPGhatguy LPGhatguy

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LPGhatguy @coveralls @ZoteTheMighty