-
Notifications
You must be signed in to change notification settings - Fork 3k
Add tool for project fields and items #1145
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
Adds a new get_project_field tool for retrieving specific project fields from GitHub projects for both users and organizations. This PR implements issue #44 by creating functionality to fetch individual project field details by field ID.
- Implements
GetProjectFieldfunction with support for user and org projects - Adds comprehensive test coverage for the new tool including error cases
- Updates existing
list_project_fieldstool to use correct numeric type for project numbers
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/projects.go | Adds GetProjectField function and fixes projectNumber type in ListProjectFields |
| pkg/github/projects_test.go | Adds comprehensive test cases for GetProjectField tool |
| pkg/github/toolsnaps/get_project_field.snap | Tool snapshot for new get_project_field tool |
| pkg/github/toolsnaps/list_project_fields.snap | Updated snapshot reflecting projectNumber type change |
| README.md | Updated documentation to reflect projectNumber type change |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
848c5c1 to
a3d585d
Compare
a3d585d to
e76e309
Compare
6112541 to
004edc0
Compare
…ols' into add-discussion-tools
…ols' into add-discussion-tools
…ols' into add-discussion-tools
…ols' into add-discussion-tools
…ols' into add-discussion-tools
kerobbi
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.
Just a non blocking nit, everything else looks solid!
2019984 to
32d7809
Compare
* Add get project fields tool * Data types * Docs * Update projectNumber's type * Add list_project_items tool * Add get_project_item tool * Return minimal project * Remove unused per_page
This pr adds:
get_project_fieldtool for user and orgget_project_itemtool for user and orglist_project_itemstool for user and orgUpdates:
projectNumbertool param toproject_numberAfter this PR is merged last chunk of project tools to be added are:
add_project_itemdelete_project_itemupdate_project_itemIt's worth noting that we are returning minimal project v2 and not the full response.
Implements #44