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

Fix "Cannot read property text of undefined" #816

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

glazar
Copy link

@glazar glazar commented Jun 12, 2017

Same code changes as found here istanbuljs-archived-repos/istanbul-reports@764cec0

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.523% when pulling dec67e3 on glazar:structuredTextFix into bc84c31 on gotwarlost:master.

@michaelBenin
Copy link

My team could definitely use this PR. Thanks @glazar - hope it gets merged soon.

@glazar glazar mentioned this pull request Jun 29, 2017
@deibeljc
Copy link

This is currently needed for us, is there any update on the status of this?

@buenjybar
Copy link

any status update on this ?

@jacksonrayhamilton
Copy link

@gotwarlost is this a sufficient and mergeable solution?

@DonkeyKongJr
Copy link

Figured out the same problem for us today. Any news on this?

@po5i
Copy link

po5i commented Feb 21, 2018

This should be merged on master. Please @gotwarlost

@MathRobin
Copy link

Hi, any news about merge ?

Having the same problem targeted by the fix. Need to see it merged :s

(thanks for this wonderful tool)

@po5i
Copy link

po5i commented Mar 26, 2018

I had to stick it to the commit at my package.json file. Let's wait until it's merged...

@jacksonrayhamilton
Copy link

R.I.P. @gotwarlost. If only he had willed this repository to a new maintainer…

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.

9 participants