- Notifications
You must be signed in to change notification settings - Fork2.7k
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
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
Adds a newget_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
GetProjectField
function with support for user and org projects - Adds comprehensive test coverage for the new tool including error cases
- Updates existing
list_project_fields
tool 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 orlearn how to get started.
Uh oh!
There was an error while loading.Please reload this page.
848c5c1
toa3d585d
Comparea3d585d
toe76e309
CompareUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
6112541
to004edc0
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
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!
Uh oh!
There was an error while loading.Please reload this page.
2019984
to32d7809
Compare0885601
intomainUh oh!
There was an error while loading.Please reload this page.
* 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
Uh oh!
There was an error while loading.Please reload this page.
This pr adds:
get_project_field
tool for user and orgget_project_item
tool for user and orglist_project_items
tool for user and orgUpdates:
projectNumber
tool param toproject_number
After this PR is merged last chunk of project tools to be added are:
add_project_item
delete_project_item
update_project_item
It's worth noting that we are returning minimal project v2 and not the full response.
Implements#44