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

Let #getRetries() return an Integer instead of int #67

Merged
merged 1 commit into from
Jun 21, 2018

Conversation

rtzll
Copy link

@rtzll rtzll commented Jun 21, 2018

Since retries are optional it may never have been set, meaning the field
could be null.
In order to avoid a NullPointerException the return type Integer is
used.

Since retries are optional it may never have been set, meaning the field
could be null.
In order to avoid a NullPointerException the return type Integer is
used.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 56.285% when pulling 1500bce on rtzll:optional-retries-as-integer into ba983eb on ChargeTimeEU:master.

@codecov-io
Copy link

Codecov Report

Merging #67 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #67   +/-   ##
=========================================
  Coverage     53.49%   53.49%           
  Complexity      812      812           
=========================================
  Files           180      180           
  Lines          3047     3047           
  Branches        211      211           
=========================================
  Hits           1630     1630           
  Misses         1332     1332           
  Partials         85       85

@TVolden TVolden merged commit 1debb7e into ChargeTimeEU:master Jun 21, 2018
@TVolden
Copy link
Member

TVolden commented Jun 21, 2018

Thanks, it's merged.

@kannan00008
Copy link

kannan00008 commented Jun 25, 2018 via email

@TVolden
Copy link
Member

TVolden commented Jun 25, 2018

Hi @kannan00008

I'm not sure what you mean, so please forgive me if I'm way off.

If you want to implement a central system or a client, then please see the start guide:
https://github.com/ChargeTimeEU/Java-OCA-OCPP/wiki/Getting-started

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.

6 participants