-
Notifications
You must be signed in to change notification settings - Fork 11
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
Allow to control code coverage #50
Conversation
45229e4
to
d30e8f8
Compare
d30e8f8
to
f38fe66
Compare
This is now ready for review. |
@webknjaz any chance that you could review this? |
#54 should fix broken CI. |
Co-authored-by: Sviatoslav Sydorenko <[email protected]>
@webknjaz thanks for reviewing and merging! |
Any chance there will be a new release that includes this option published soon? |
Contains #49. Will rebase once that is merged.(That is now done.)Provides a new option
coverage
with valuesalways
,never
, andauto
. The behavior forauto
is to not send code coverage whenpull-request-change-detection=true
and the action is called from a PR; this can be adjusted later (ref: #46 (comment), #46 (comment)).