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.

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

Modify comment syntax for code coverage #22752

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Conversation

jatgarg
Copy link
Contributor

@jatgarg jatgarg commented Oct 8, 2024

Description

Modify comment for code coverage.
Add the line coverage summary to the summary.

@jatgarg jatgarg self-assigned this Oct 8, 2024
@github-actions github-actions bot added area: build Build related issues base: main PRs targeted against main branch labels Oct 8, 2024
@jatgarg jatgarg requested a review from tylerbutler October 8, 2024 19:43
@jatgarg jatgarg enabled auto-merge (squash) October 8, 2024 19:58
@jatgarg jatgarg requested a review from alexvy86 October 8, 2024 21:40
Copy link
Member

@tylerbutler tylerbutler left a comment

Choose a reason for hiding this comment

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

Looks good overall, but check the sort logic.

@jatgarg jatgarg requested a review from tylerbutler October 8, 2024 22:03
@jatgarg jatgarg merged commit 7a7263c into microsoft:main Oct 8, 2024
26 checks passed
chentong7 pushed a commit to chentong7/FluidFramework that referenced this pull request Oct 8, 2024
## Description

Modify comment for code coverage.
Add the line coverage summary to the summary.

Co-authored-by: Jatin Garg <jatingarg@Jatins-MacBook-Pro-2.local>
sonalideshpandemsft pushed a commit that referenced this pull request Oct 9, 2024
## Description

Modify comment for code coverage.
Add the line coverage summary to the summary.

Co-authored-by: Jatin Garg <jatingarg@Jatins-MacBook-Pro-2.local>
@jatgarg jatgarg deleted the fixcolor branch October 10, 2024 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: build Build related issues base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants