Skip to content

Conversation

steinfurt
Copy link

@steinfurt steinfurt commented Aug 18, 2025

Fixes microsoft/vscode#256731
It also addresses most of microsoft/vscode#238255

@steinfurt
Copy link
Author

@microsoft-github-policy-service agree

The most important information is the branch name which can be helpful
to find out what the implemented feature is about or to extract ticket
numbers to be referenced in the commit message.

GitHub information about the repository owner/name and possible pull
request information is optional.

The implementation is based on [1].

[1] https://github.com/microsoft/vscode-copilot-chat/blob/aa45bf559489cc5d21e93bd5cce7388056819222/src/extension/prompts/node/agent/agentPrompt.tsx#L497-L524
@steinfurt steinfurt force-pushed the repo-context-for-commit-message branch from bfa8151 to 0281130 Compare August 18, 2025 18:25
@steinfurt
Copy link
Author

steinfurt commented Sep 21, 2025

Hello @lszomoru,

I hope you don't mind me asking here about the status of this PR. Obviously, I know one has to be patient when submitting an open source contribution. You've probably had more than enough other topics to take care of in recent weeks. Still, I wanted to quickly check in if there is anything missing, anything to be changed here or if you've got any other kind of feedback.

I originally believed this PR to almost be a no-brainer, given that the linked issue has been created by a VS code team member, there is another issue with quite a few upvotes, the missing implementation has been discussed in the AMA of the v1.102 release and this just adds the suggested implementation in a minimal way (though not re-using the existing component).

I don't want to be impolite, but after a month I thought it would be okay to ask for any kind of reaction... Obviously, no problem if it takes a little longer...

@vs-code-engineering vs-code-engineering bot added this to the September 2025 milestone Sep 22, 2025
@lszomoru lszomoru self-requested a review September 22, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include repo / branch context in Generate Commit Message tool
3 participants