Skip to content

Add missing milestone to create_issue - #88

Merged
williammartin merged 2 commits into
mainfrom
wm/missing-milestone
Apr 4, 2025
Merged

Add missing milestone to create_issue#88
williammartin merged 2 commits into
mainfrom
wm/missing-milestone

Conversation

@williammartin

Copy link
Copy Markdown
Collaborator

Description

Fixes#78

CopilotAI review requested due to automatic review settings April 3, 2025 20:13

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 support for a missing milestone parameter in the create_issue functionality to fix an issue with milestone handling. Key changes include:

  • Adding "milestone" to the tool's input schema in tests.
  • Updating the mock issue to include a milestone value.
  • Incorporating an optional milestone parameter in the issue creation logic.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

FileDescription
pkg/github/issues_test.goAdds assertions and test cases for milestone.
pkg/github/issues.goIntegrates optional milestone parameter into issue creation.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

Comment threadpkg/github/issues.go
Comment threadpkg/github/issues.go Outdated
),
),
mcp.WithNumber("milestone",
mcp.Description("New milestone number"),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it new or can it be an existing milestone number that attaches the new issue to the existing milestone?

Base automatically changed from wm/boundaries to mainApril 3, 2025 21:34
@williammartin
williammartin merged commit 95c7b0a into mainApr 4, 2025
@williammartin
williammartin deleted the wm/missing-milestone branch April 4, 2025 12:47
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
* Add: Needle MCP Server (search, rag)
* fix: update image & description
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

create_issue is missing milestone

3 participants

@williammartin@SamMorrowDrums