Skip to content

Conversation

dlplenin
Copy link
Contributor

@dlplenin dlplenin commented Aug 2, 2019

@MarcoRossignoli
Average calculation: This PR is related with #479

Today I installed a nightly build of coverlet.msbuild and I realized that a piece of code in coverlet.msbuild.tasks was not taken into account in the previous PR, to keep consistency.

Copy link
Collaborator

@MarcoRossignoli MarcoRossignoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks
Oh yep...duplicated code issue 😞 I missed that.

@MarcoRossignoli
Copy link
Collaborator

@MarcoRossignoli MarcoRossignoli merged commit 83e93db into coverlet-coverage:master Aug 3, 2019
@MarcoRossignoli
Copy link
Collaborator

@dlplenin Diego can you try tomorrow and confirm correct output please?

@dlplenin
Copy link
Contributor Author

dlplenin commented Aug 3, 2019

@dlplenin Diego can you try tomorrow and confirm correct output please?

Monday for sure! Now I dont have my computer with me, sorry.
I'll keep u posted.

@MarcoRossignoli
Copy link
Collaborator

MarcoRossignoli commented Aug 3, 2019

Oh yes I mean "tomorrow" we'll have new nightly...when you can 😄

@dlplenin
Copy link
Contributor Author

dlplenin commented Aug 5, 2019

@MarcoRossignoli
It's working as expected! Thanks 👍

Running a docker compose:
docker-compose-coverage-

@MarcoRossignoli
Copy link
Collaborator

Thank's to you for help!

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.

3 participants