Skip to content
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

Add /diff to the assistant #24395

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ross39
Copy link

@ross39 ross39 commented Feb 6, 2025

Release Notes:

Not sure if this fully closes #16821 but a basic git diff has been added to the assistant panel. As this is my first PR and I am new, any feedback would be appreciated. I am happy to make this an extension if the zed team thinks an extension would be a better place for this. Thanks! 👋

Below are screenshots of what it looks like in the assistant panel.

How the command looks in the assistant panel
Screenshot 2025-02-07 at 23 48 40

How the diff looks when added to the context
Screenshot 2025-02-07 at 23 48 51

This comment was marked as resolved.

@ross39

This comment was marked as resolved.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 6, 2025

This comment was marked as resolved.

@zed-industries-bot
Copy link

Warnings
⚠️

This PR is missing release notes.

Please add a "Release Notes" section that describes the change:

Release Notes:

- Added/Fixed/Improved ...

If your change is not user-facing, you can use "N/A" for the entry:

Release Notes:

- N/A

Generated by 🚫 dangerJS against 04342c0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add /diff to the assistant
2 participants