- Notifications
You must be signed in to change notification settings - Fork1.3k
add author field to list_commits for filtering#569
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
add author field to list_commits for filtering#569
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this 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 pull request adds an optional "author" field to the list_commits API for filtering commits based on author information.
- Adds the "author" property to the input schema in tests.
- Updates the repositories function to retrieve and pass the new "author" parameter.
- Updates snapshots and documentation in README to reflect the new parameter.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
File | Description |
---|---|
pkg/github/repositories_test.go | Adds tests to check for the new "author" property in schema and request. |
pkg/github/repositories.go | Updates the input schema and passes "author" to the GitHub API. |
pkg/github/toolsnaps/list_commits.snap | Updates the snapshot to include the "author" field. |
README.md | Documents the new optional "author" parameter. |
f631ff5
intogithub:mainUh oh!
There was an error while loading.Please reload this page.
pasevin commentedJun 29, 2025
Thanks! Any idea when this is going to be deployed tohttps://api.githubcopilot.com/mcp/? |
@pasevin expecting to cut a new semver release and deploy to the remote server this week (likely Tuesday). |
Closes:#568