Skip to content
This repository was archived by the owner on Feb 20, 2019. It is now read-only.

WIP: Add an "add" function #59

Merged
merged 3 commits into from
Dec 19, 2016

Conversation

valishah
Copy link
Contributor

Hi , This is my first open source pull request . Seem like everything went good. While doing commit i have got stuck with code coverage.

  1. add › Adds two integers
    failed with "(0 , _src.add) is not a function"

Hope to contribute more.

@codecov-io
Copy link

codecov-io commented Dec 19, 2016

Current coverage is 100% (diff: 100%)

Merging #59 into master will not change coverage

@@           master   #59   diff @@
===================================
  Files          15    16     +1   
  Lines          51    52     +1   
  Methods         0     0          
  Messages        0     0          
  Branches        0     0          
===================================
+ Hits           51    52     +1   
  Misses          0     0          
  Partials        0     0          

Powered by Codecov. Last update 031ab8d...65b66fe

Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

Looks great! Just one small thing. Thanks for contributing!

@@ -30,4 +31,5 @@ export {
isNullOrWhitespace,
startsWith,
removeDuplicates,
additon,
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer that this were called add instead of addition. Care to make the update?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great. Sure, i would do that. Thanks a lot for quick response.

@valishah
Copy link
Contributor Author

I have updated the PR .

@kentcdodds
Copy link
Member

Terrific! Thanks!

@kentcdodds kentcdodds merged commit 8a8a9d9 into eggheadio-github:master Dec 19, 2016
@valishah
Copy link
Contributor Author

Thanks for your help @kentcdodds :) Can you share the url for knowing what open source projects to contribute ?

@kentcdodds
Copy link
Member

Yeah, check this out :-)

@valishah
Copy link
Contributor Author

Thanks a lot @kentcdodds . I will be helping the people who comes here to learn what i have learned. Thanks again for all your efforts. I really appreciate that.

@kentcdodds
Copy link
Member

Awesome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants