Skip to content

Conversation

@jtreminio
Copy link
Contributor

Closes #191

@codecov-io
Copy link

codecov-io commented Dec 17, 2019

Codecov Report

Merging #192 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #192      +/-   ##
============================================
- Coverage     93.19%   93.18%   -0.01%     
  Complexity      486      486              
============================================
  Files            27       27              
  Lines          1470     1468       -2     
============================================
- Hits           1370     1368       -2     
  Misses          100      100
Impacted Files Coverage Δ Complexity Δ
src/Builder/CliMenuBuilder.php 83.58% <ø> (-0.09%) 65 <0> (ø)
src/Builder/SplitItemBuilder.php 73.52% <ø> (-0.76%) 10 <0> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c4db90...240a745. Read the comment docs.

@AydinHassan
Copy link
Member

Let's go with this. Thanks @jtreminio .

@AydinHassan AydinHassan merged commit 4618e39 into php-school:master Dec 17, 2019
@jtreminio jtreminio mentioned this pull request Dec 17, 2019
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.

Don't call $callback->bindTo()

3 participants