Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[PublishCodeCoverageResultsV1] Apply restrictions #14512

Merged
merged 4 commits into from
Apr 5, 2021
Merged

[PublishCodeCoverageResultsV1] Apply restrictions #14512

merged 4 commits into from
Apr 5, 2021

Conversation

DaniilShmelev
Copy link
Contributor

@DaniilShmelev DaniilShmelev commented Mar 2, 2021 •
edited
Loading

Task name: PublishCodeCoverageResults

Documentation changes required: No

Added unit tests: No

Checklist:

@DaniilShmelev DaniilShmelev requested a review from a team March 2, 2021 17:37
@DaniilShmelev DaniilShmelev self-assigned this Mar 2, 2021
@DaniilShmelev
Copy link
Contributor Author

Note: can't apply restricted mode here since this task uses [codecoverage.publish] command

@anatolybolshakov
Copy link
Contributor

LGTM, thanks! Could you please update V2 also?

@DaniilShmelev
Copy link
Contributor Author

PublishCodeCoverageV2 is not included in the default collection currently, no need to update it

@EzzhevNikita EzzhevNikita requested a review from a team March 5, 2021 14:50
@EzzhevNikita EzzhevNikita merged commit 502c284 into master Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EzzhevNikita EzzhevNikita EzzhevNikita approved these changes

@anatolybolshakov anatolybolshakov anatolybolshakov approved these changes

@nigurr nigurr Awaiting requested review from nigurr

@SadagopanRajaram SadagopanRajaram Awaiting requested review from SadagopanRajaram

@ganesp ganesp Awaiting requested review from ganesp

@nadesu nadesu Awaiting requested review from nadesu

Assignees

@DaniilShmelev DaniilShmelev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DaniilShmelev @anatolybolshakov @EzzhevNikita