Skip to content

Allow to load ".set" files, which aggregate a set of coverage files - #8

Open
zxgio wants to merge 2 commits into
rizinorg:masterfrom
zxgio:set-operations-on-coverages
Open

Allow to load ".set" files, which aggregate a set of coverage files#8
zxgio wants to merge 2 commits into
rizinorg:masterfrom
zxgio:set-operations-on-coverages

Conversation

@zxgio

@zxgio zxgio commented May 7, 2020

Copy link
Copy Markdown

This patch allows loading the result of the intersection/union/subtraction of a set of coverage files. A (text) ".set" file consists of a line with the desired operations and the list of coverage files to aggregate.

@oddcoder

oddcoder commented May 8, 2020

Copy link
Copy Markdown
Collaborator

can you please add a unit test for that ?

@oddcoder

Copy link
Copy Markdown
Collaborator

ping @zxgio

@zxgio

zxgio commented May 10, 2020

Copy link
Copy Markdown
Author

Hi @oddcoder , sorry for the delay; in a couple of weeks, I'll try to refactor my code and add unit tests. Now I'm working overtime for a job-related deadline.

@oddcoder

Copy link
Copy Markdown
Collaborator

ping @zxgio

@zxgio

zxgio commented Jul 11, 2020

Copy link
Copy Markdown
Author

I have refactored my code and added some tests. Locally it seems fine, I can't see what's wrong with the "Build phase" (I've never used Google Cloud Build)

@oddcoder

Copy link
Copy Markdown
Collaborator

The google cloud thing is unrelated, I am in the middle of exams, will finish my exams and check, thanks in advance.

@zxgio

zxgio commented Aug 2, 2020

Copy link
Copy Markdown
Author

No problem, thanks!

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