-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Add tools for Projects V2 #1114
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
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 PR adds a new get_project
tool to retrieve a specific project by number and removes pagination parameters from the existing list_projects
tool.
- Implements
GetProject
function to fetch a single project by number for users or organizations - Removes
before
andafter
pagination parameters fromListProjects
tool - Updates documentation and tests to reflect these changes
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
pkg/github/tools.go | Registers the new GetProject tool in the projects toolset |
pkg/github/projects.go | Implements GetProject function and removes pagination from ListProjects |
pkg/github/projects_test.go | Adds comprehensive tests for GetProject and updates ListProjects tests |
pkg/github/toolsnaps/list_projects.snap | Updates tool snapshot without pagination parameters |
pkg/github/toolsnaps/get_project.snap | New tool snapshot for GetProject |
README.md | Updates documentation to include GetProject and remove pagination from ListProjects |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
430c552
to
62bcbfc
Compare
"readOnlyHint": true | ||
}, | ||
"description": "List Projects for a user or organization", | ||
"description": "List Projects for an user or org", |
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.
it would be 'a user or organisation' 🫢
3d20042
to
31000b0
Compare
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.
LGTM!
* Add get_project tool * Remove pagination for now * Fix url formatting * Minor tweaks * Add list project fields tool * Wording
This pr:
list_projects
toolget_project
tool for user and orglist_project_fields
tool for user and orgImplements #44
Implements new projects V2 API. FYI implementation of project tools will use google/go-github when they implement the new API.
Pagination will be added in the next step.