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

Cody: Fix stop generating error #1668

Merged
merged 3 commits into from
Nov 13, 2023
Merged

Cody: Fix stop generating error #1668

merged 3 commits into from
Nov 13, 2023

Conversation

umpox
Copy link
Contributor

@umpox umpox commented Nov 7, 2023

Description

There is a case where a user aborts the chat before a message is sent. In that case we don't actually want an error. We can handle this for each individual case.

We now already do this for fixups, which this was originally added for:

https://sourcegraph.com/github.com/sourcegraph/cody@tr/fix-stop-gen-error/-/blob/vscode/src/chat/FixupViewProvider.ts?L101%3A1-104%3A10

image

Test plan

  • Check no error when "Stop generating" is clicked
  • Check no error when fixup is generated with incorrect xml tag

@umpox umpox requested a review from abeatrix November 7, 2023 14:30
Copy link
Contributor

@abeatrix abeatrix left a comment

Choose a reason for hiding this comment

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

Thank you! Worth adding to changelog?

@umpox umpox enabled auto-merge (squash) November 7, 2023 14:36
@umpox umpox merged commit 15b1dbb into main Nov 13, 2023
14 checks passed
@umpox umpox deleted the tr/fix-stop-gen-error branch November 13, 2023 09:01
@umpox umpox self-assigned this Nov 30, 2023
Sign up for free to 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.

None yet

2 participants