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.

Sign up for GitHub

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

Jump to bottom

Backport 7.0.4 fixes to 7.0.2 #2322

Closed
wants to merge 3 commits into from
Closed

Backport 7.0.4 fixes to 7.0.2 #2322

wants to merge 3 commits into from
+61 −53

Commits on Sep 19, 2024

  1. Switch back to cody main (#2270) 

    ## Test plan

Full QA
    @pkukielka
    pkukielka committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    f610529 View commit details
    Browse the repository at this point in the history

  2. Fix chat history download (#2282) 

    Fixes #2274

## Test plan

1. Open Cody chat history
2. Click "Export History"
3. Select a file and path of your choice to save it
4. Verify that file content contains proper json
    @pkukielka
    pkukielka committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    3c38578 View commit details
    Browse the repository at this point in the history

  3. Do not run EDT code in CodyFixHighlightPass:doCollectInformation (#2289) 

    Fixes #2287

## Changes 

Looks like we cannot do EDT actions in the
`CodyFixHighlightPass:doCollectInformation` or we are risking a
deadlock.

## Test plan

It's a bit hard to test deterministically but for me adding and removing
few special characters on an empty file was triggering IDE crash pretty
quickly (10-20 sec).
I used combination of:

`Option + \` (`«`)
`Option + P` (`Ļ`)
`Backspace`

In a random order.
Timing matters, keep changing the tempo of the keystrokes.
    @pkukielka
    pkukielka committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    d1068e2 View commit details
    Browse the repository at this point in the history