Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Remove non-existent get_line_number_in_pull_request_file tool reference#436

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

Open
TheoBrigitte wants to merge1 commit intogithub:main
base:main
Choose a base branch
Loading
fromTheoBrigitte:remove-get_line_number_in_pull_request_file

Conversation

TheoBrigitte
Copy link

Closes:#435

Theadd_pull_request_review_comment_to_pending_review tool mention theget_line_number_in_pull_request_file tool to be used to get line number. But this tool does not exists.

mcp.WithDescription(t("TOOL_ADD_PULL_REQUEST_REVIEW_COMMENT_TO_PENDING_REVIEW_DESCRIPTION","Add a comment to the requester's latest pending pull request review, a pending review needs to already exist to call this (check with the user if not sure). If you are using the LINE subjectType, use the get_line_number_in_pull_request_file tool to get an exact line number before commenting.")),

Image

@CopilotCopilotAI review requested due to automatic review settingsMay 25, 2025 18:57
@TheoBrigitteTheoBrigitte requested a review froma team as acode ownerMay 25, 2025 18:57
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Removes a reference to the non-existentget_line_number_in_pull_request_file tool from the pull request review comment helper and reorders thetranslations import into the internal imports block.

  • Deleted the misleading tool recommendation from the description ofadd_pull_request_review_comment_to_pending_review
  • Moved thetranslations import to the internal imports section for clarity

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Non existent get_line_number_in_pull_request_file tool
1 participant
@TheoBrigitte

[8]ページ先頭

©2009-2025 Movatter.jp