Skip to content

experimental/stats/metricregistry: Add comments on enum consts for Metrics Type #7457

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

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

zasweq
Copy link
Contributor

@zasweq zasweq commented Jul 30, 2024

The PR fixes a linter in Google 3.

RELEASE NOTES: N/A

@zasweq zasweq requested a review from arvindbr8 July 30, 2024 16:54
@zasweq zasweq added the Type: Documentation Documentation or examples label Jul 30, 2024
@zasweq zasweq added this to the 1.66 Release milestone Jul 30, 2024
@arvindbr8 arvindbr8 assigned zasweq and unassigned arvindbr8 Jul 30, 2024
Copy link

codecov bot commented Jul 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.47%. Comparing base (0b33bfe) to head (d8328b8).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7457      +/-   ##
==========================================
- Coverage   81.49%   81.47%   -0.02%     
==========================================
  Files         354      354              
  Lines       27078    27078              
==========================================
- Hits        22066    22062       -4     
  Misses       3815     3815              
- Partials     1197     1201       +4     
Files Coverage Δ
experimental/stats/metricregistry.go 94.52% <ø> (ø)

... and 15 files with indirect coverage changes

@zasweq zasweq merged commit 5520cff into grpc:master Jul 30, 2024
13 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Documentation Documentation or examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants