- 
                Notifications
    You must be signed in to change notification settings 
- Fork 2.9k
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
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