-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/add multiple threshold #379 #1123
Feat/add multiple threshold #379 #1123
Conversation
@MarcoRossignoli can you please validate? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes and here we go!
Thanks!
@pbmiguel seems that rebase didn't go well...I see 22 files changed you've included some old commits. |
3d665db
to
356bf27
Compare
@MarcoRossignoli, it's fixed now. You can see the new changes here |
356bf27
to
b8111fc
Compare
b8111fc
to
99cb653
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
thanks @pbmiguel! |
@MarcoRossignoli, when will you expected this to be released? because the last release was in February |
We don't have ETA yet, need to schedule, I'll take a look at issue list to understand if we want more fix before release, anyway you can dogfood our nighty build if you want, so you'll help us with testing also https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md |
Me and @FranciscoSousaDeveloper tried to continue this Pull Request but we had no permission. So this Pull Request is the continuation of #385 with rebase and updated documentation.
Basically, this Pull Request enables the specification of multiple threshold values.