Partial Update Support about ruby-codacy-coverage HOT 3 CLOSED

You are right, I got a bit confused during the answer. That looks good.

from ruby-codacy-coverage.

@mtamhankar1 at the moment only https://github.com/codacy/codacy-coverage-reporter supports partial coverage.

Regarding the guidance I think your idea is nice I would maybe use a parameter in the cli instead of an environment variable to identify that it was a partial.

The final request would need in some way to be "centralised", so you can add a command to this library or just another flag that can be invoked when all the partials are done.

Not sure if answered your questions. Let me know what you think.

from ruby-codacy-coverage.

Regarding the guidance I think your idea is nice I would maybe use a parameter in the cli instead of an environment variable to identify that it was a partial.

I am still perhaps a little bit confused on this piece, since the usage is not in the cli but rather is invoked in spec_helper.rb as Codacy::Reporter.start. So is the suggestion here to add a parameter such that a user could instead call something like Codacy::Reporter.start(partial: true)?

The final request would need in some way to be "centralised", so you can add a command to this library or just another flag that can be invoked when all the partials are done.

I can certainly add a command for the final request, I think this makes the most sense.

from ruby-codacy-coverage.