Tobias Pfeiffer comments

Results 513 comments of


                                            Tobias Pfeiffer

Allow Overall Statistics Calculation without loading the files that are covered

Hi there, the major technical reason is that the goal of SimpleCov is to show which lines are covered and which aren't - having some overall statistics is fine but...

Allow Overall Statistics Calculation without loading the files that are covered

It just occurred to me that the reason we are reading all files is because we need to look for the nocov comments to accurately report coverage. If we were...

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

Hi there @chase-stevens, thanks for the report :green_heart: The interesting question for me would be why it doesn't have access to these files any more? Seems like there might be...

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

@chase-stevens :wave: hi there, any update on this and the questions from your side? :)

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

Closing due to missing feedback

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

@felixscheinost what is "Nix" ? I'm not sure whether this is another problem we should fix or that users should fix... like we already integrate with so much. Us messing...

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

@felixscheinost have you found a user level solution yet? While I generally would like to have things working everywhere, doing this would introduce some overhead. Of course, if we switched...

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

I was just thinking, we might introduce asset inlining which would also solve this problem - right?

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

@slinkp maybe :shrug: Couldn't do it like this because it deletes the whole path. People might (accidentally) throw files in there and we can't just delete them all. Could we...

Simplecov cannot generate a new coverage report because it does not skip writing already existing asset files.

Hey, thanks to feedback here I thought about it some more and theoretically I guess we could: * read the files and just write them as new files to avoid...