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

Go: automated mad coverage report #14445

Merged
merged 5 commits into from
Oct 15, 2023
Merged

Go: automated mad coverage report #14445

merged 5 commits into from
Oct 15, 2023

Conversation

owen-mc
Copy link
Contributor

@owen-mc owen-mc commented Oct 11, 2023 •
edited
Loading

Set up automated coverage reports for Go

@owen-mc owen-mc requested a review from tamasvajk October 11, 2023 15:41
@github-actions github-actions bot added the Go label Oct 11, 2023
@owen-mc owen-mc force-pushed the go/automated-mad-coverage-report branch from d9febde to dfcdb4a Compare October 11, 2023 20:11
@owen-mc owen-mc marked this pull request as ready for review October 11, 2023 20:11
@owen-mc owen-mc requested review from a team as code owners October 11, 2023 20:11
smowton
smowton previously approved these changes Oct 11, 2023
View reviewed changes
Copy link
Contributor

@smowton smowton left a comment

Choose a reason for hiding this comment

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

Looks plausible

@owen-mc owen-mc added the no-change-note-required This PR does not need a change note label Oct 11, 2023
tamasvajk
tamasvajk approved these changes Oct 12, 2023
View reviewed changes
Copy link
Contributor

@tamasvajk tamasvajk left a comment

Choose a reason for hiding this comment

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

LGTM

@owen-mc owen-mc merged commit 5356100 into github:main Oct 15, 2023
19 checks passed
@owen-mc owen-mc deleted the go/automated-mad-coverage-report branch October 15, 2023 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tamasvajk tamasvajk tamasvajk approved these changes

@smowton smowton smowton left review comments

Assignees
No one assigned
Labels
C# Go Java no-change-note-required This PR does not need a change note
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@owen-mc @smowton @tamasvajk