Skip to content

Conversation

@erhebend-tai
Copy link

We are excited to review your PR.

So we can do the best job, please check:

  • There's a descriptive title that will make sense to other developers some time from now.
  • There's associated issues. All PR's should have issue(s) associated - unless a trivial self-evident change such as fixing a typo. You can use the format Fixes #nnnn in your description to cause GitHub to automatically close the issue(s) when your PR is merged.
  • Your change description explains what the change does, why you chose your approach, and anything else that reviewers should know.
  • You have included any necessary tests in the same PR.

Copilot AI review requested due to automatic review settings November 4, 2025 04:49
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.

Pull Request Overview

This PR improves the clarity of documentation in the Fairlearn metrics module by correcting grammatical usage in the interface documentation. The change updates "which key" to "where key" for better grammatical accuracy.

  • Corrected grammatical phrasing in XML documentation comment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ericstj
Copy link
Member

ericstj commented Nov 5, 2025

This PR is only changing internal docs. While it does correct a grammatical problem, the PR is not a good use of resources or time. Please refrain from submitting these trivial PRs.

@ericstj ericstj closed this Nov 5, 2025
@erhebend-tai
Copy link
Author

erhebend-tai commented Nov 5, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants