-
Notifications
You must be signed in to change notification settings - Fork 3k
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
GetProjectfunction to fetch a single project by number for users or organizations - Removes
beforeandafterpagination parameters fromListProjectstool - 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
MattBabbage
left a comment
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_projectstoolget_projecttool for user and orglist_project_fieldstool 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.