Skip to content

Get rid off "JSON.parse(JSON.dump(...))" #916

Open
@PragTob

Description

@PragTob

Twice in the code you will find JSON.parse(JSON.dump which is basically used as a way to normalize the coverage data read from file with the one we collect in process. This went in during the frantic work on branch coverage.

A mechanism that is more lightweight in making those uniform would be welcome. Of course, it might be this is the most efficient way to do this but I doubt it ;)

I'm not sure how to go about this, aka if it's best to just adapt our internal data structures to be like the read/parsed ones or the other way around. I'm not even sure what exactly the problem/difference is so looking at that might be a good first step.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions