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

check for nil account IDs in get user #124

Merged
merged 2 commits into from
May 2, 2020

Conversation

StephenButtolph
Copy link
Contributor

@StephenButtolph StephenButtolph commented May 2, 2020

Check for nil Account IDs in utility functions that are called by services.

@codecov
Copy link

codecov bot commented May 2, 2020

Codecov Report

Merging #124 into master will increase coverage by 0.28585%.
The diff coverage is 81.25000%.

@@                 Coverage Diff                 @@
##              master        #124         +/-   ##
===================================================
+ Coverage   64.01056%   64.29641%   +0.28584%     
===================================================
  Files            223         223                 
  Lines          14018       14021          +3     
===================================================
+ Hits            8973        9015         +42     
+ Misses          4297        4249         -48     
- Partials         748         757          +9     

@StephenButtolph StephenButtolph merged commit f257187 into ava-labs:master May 2, 2020
@StephenButtolph StephenButtolph deleted the check-nil-account branch May 2, 2020 18:51
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.

1 participant