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

enable xdebug on hhvm #58

Merged
merged 1 commit into from
Jun 3, 2017
Merged

enable xdebug on hhvm #58

merged 1 commit into from
Jun 3, 2017

Conversation

xKerman
Copy link
Owner

@xKerman xKerman commented Jun 3, 2017

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 3, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##             master    #58   +/-   ##
=======================================
  Coverage       100%   100%           
  Complexity       29     29           
=======================================
  Files            20     20           
  Lines           176    176           
=======================================
  Hits            176    176
Flag Coverage Δ Complexity Δ
#5_2 100% <ø> (ø) 0 <ø> (ø) ⬇️
#5_3 100% <ø> (ø) 0 <ø> (ø) ⬇️
#5_4 100% <ø> (ø) 0 <ø> (ø) ⬇️
#5_5 100% <ø> (ø) 0 <ø> (ø) ⬇️
#5_6 100% <ø> (ø) 29 <ø> (ø) ⬇️
#hhvm 79.45% <ø> (?) 29 <ø> (?)

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 d5baf95...bb5f3ab. Read the comment docs.

@xKerman xKerman force-pushed the fix/hhvm-coverage branch from 20b4fcd to bb5f3ab Compare June 3, 2017 16:25
@xKerman
Copy link
Owner Author

xKerman commented Jun 3, 2017

hhvm code coverage is lower than other PHP coverage, but it seems that this is caused by facebook/hhvm#1589

@xKerman xKerman merged commit 73f7bbc into master Jun 3, 2017
@xKerman xKerman deleted the fix/hhvm-coverage branch June 3, 2017 22:14
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.

2 participants