Skip to content

Conversation

@Nyholm
Copy link
Member

@Nyholm Nyholm commented Jan 6, 2016

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

maybe:

if ($hasValue) {
    $this->value = $value;
}

@codecov-io
Copy link

Current coverage is 27.21%

Merging #4 into master will increase coverage by +1.16% as of 50d9fef

@@            master      #4   diff @@
======================================
  Files            2       2       
  Stmts          142     147     +5
  Branches         0       0       
  Methods         21      21       
======================================
+ Hit             37      40     +3
  Partial          0       0       
- Missed         105     107     +2

Review entire Coverage Diff as of 50d9fef

Powered by Codecov. Updated on successful CI builds.

Nyholm added a commit that referenced this pull request Jan 6, 2016
Added more constructor arguments
@Nyholm Nyholm merged commit dcfacaf into php-cache:master Jan 6, 2016
@Nyholm Nyholm deleted the patch-3 branch January 6, 2016 18:35
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.

3 participants