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

Add API calls that use default Context #169

Merged

ChristopherHogan merged 1 commit into master from chogan/default_context

Apr 8, 2021

Collaborator

ChristopherHogan commented Apr 7, 2021

Fixes #130.

Add overloads of Bucket and VBucket methods that use a stored default Context (set during the constructor) instead of requiring the user to always pass one in.
Pass Contexts by const reference.
Simplify tests by removing all default Context instances.

ChristopherHogan requested a review from jya-kmu

April 7, 2021 16:30

coveralls commented Apr 7, 2021 •

edited

Loading

Coverage increased (+0.2%) to 85.322% when pulling 8f69583 on chogan/default_context into 571286c on master.

jya-kmu approved these changes

View reviewed changes


          Add API calls that use default Context

8f69583

ChristopherHogan force-pushed the chogan/default_context branch from 6cf9a2f to 8f69583 Compare

April 8, 2021 12:32

ChristopherHogan merged commit 5bcd25a into master

ChristopherHogan deleted the chogan/default_context branch

April 9, 2021 16:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet