Navigation Menu

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

Added bulk method. #30

Merged
merged 1 commit into from Feb 12, 2020
Merged

Added bulk method. #30

merged 1 commit into from Feb 12, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jan 24, 2020

Fixes #29.

@coveralls
Copy link

coveralls commented Jan 24, 2020

Coverage Status

Coverage increased (+0.1%) to 98.947% when pulling fc9a0ae on minibrew:master into 30d61d1 on vrcmarcos:master.

@vrcmarcos
Copy link
Owner

Hey, thank you for that! Your code is awesome, and will bring so much value to this library!

Do you mind to write some unit tests to keep the coverage? We're avoiding decreasing it :)

@ghost ghost force-pushed the master branch 2 times, most recently from 2ddea02 to 989a434 Compare February 6, 2020 14:09
@ghost
Copy link
Author

ghost commented Feb 6, 2020

@vrcmarcos Thanks for your suggestion and your feedback! I added one unit test but for some reason the unit test coverage seems to have decreased. It's not entirely clear to me why. Could you kindly take a look at this?

Edit: Nevermind, I fixed it. :)

@ghost ghost force-pushed the master branch 2 times, most recently from c1408a1 to 74b3215 Compare February 11, 2020 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement bulk method
2 participants