test-coverage icon indicating copy to clipboard operation
test-coverage copied to clipboard

Null-safety support

Open Albert221 opened this issue 4 years ago • 9 comments

Hi! What's the status on supporting the null-safety within this package?

Albert221 avatar Jan 05 '21 16:01 Albert221

Had a quick peek at updating it, but it seems like we need to wait for a null-safety update from coverage first

QuirijnGB avatar Feb 02 '21 03:02 QuirijnGB

covergae now has NNBD support.

lcov is no longer continued. Means to support this to NNBD lcov needs to be replaced or forked and migrated first.

masseelch avatar Mar 02 '21 06:03 masseelch

Hi! Has there been any progress around what to do about lcov?

gilescm avatar May 20 '21 13:05 gilescm

You can use dart test with the --coverage option and then format output files to lcov just like here:

https://github.com/Albert221/FastShopping/blob/5af8fc848d993d74cf0e989473f54ed112de8a00/.github/workflows/test.yml#L95-L96

Albert221 avatar May 20 '21 14:05 Albert221

I have cloned this project locally and make it as null-safety version, is it nessesary to send a merge request?

wustrong avatar Jan 29 '22 07:01 wustrong

Hi! Has there been any progress around what to do about lcov?

lcov_dart instead

wustrong avatar Jan 29 '22 07:01 wustrong

I have cloned this project locally and make it as null-safety version, is it nessesary to send a merge request?

can you send me the link so I can test your fork in my app?

AleksandarSavic95 avatar Feb 23 '22 16:02 AleksandarSavic95

I have cloned this project locally and make it as null-safety version, is it nessesary to send a merge request?

can you send me the link so I can test your fork in my app?

here is it: https://github.com/wustrong/test-coverage

wustrong avatar Jul 07 '22 07:07 wustrong

Wow, thanks! It's been a long time since I asked this! @wustrong

AleksandarSavic95 avatar Jul 11 '22 10:07 AleksandarSavic95