Skip to content

Conversation

@Rick-Anderson
Copy link
Contributor

@Rick-Anderson Rick-Anderson commented Apr 3, 2025

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (4)

aspnetcore/log-mon/metrics/built-in/includes/built-in10.md:441

  • Typo detected: 'occured' should be corrected to 'occurred'.
…If an authentication error or exception occured.

aspnetcore/log-mon/metrics/built-in/includes/built-in10.md:467

  • Typo detected: 'occured' should be corrected to 'occurred'.
…If an authentication error or exception occured.

aspnetcore/log-mon/metrics/built-in/includes/built-in10.md:480

  • Typo detected: 'occured' should be corrected to 'occurred'.
…If an authentication error or exception occured.

aspnetcore/log-mon/metrics/built-in/includes/built-in10.md:493

  • Typo detected: 'occured' should be corrected to 'occurred'.
…If an authentication error or exception occured.

@Rick-Anderson
Copy link
Contributor Author

@MackinnonBuck I copy/pasted your issue table and reviewed the commit changes to verify I resolved all the problems you identified.

Copy link
Member

@MackinnonBuck MackinnonBuck left a comment

Choose a reason for hiding this comment

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

Looks great!

@Rick-Anderson Rick-Anderson merged commit a8f21ec into main Apr 9, 2025
3 checks passed
@Rick-Anderson Rick-Anderson deleted the 10/metrics/1 branch April 9, 2025 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add AuthN/AuthZ metrics to built in metrics doc

3 participants