Skip to content

Conversation

@karthiknadig
Copy link
Member

No description provided.

karthiknadig and others added 5 commits October 7, 2024 11:10
…endtext. (#24248)

For some reason, it seems like shell integration may take awhile to get
activated: #24239

Performance issue seemed to indicate exactly 3 second coming from the
https://github.com/microsoft/vscode-python/pull/24078/files#diff-5290f3097d5f92e3495c8abfbe095dff83c3f8de3dcac08ab2d0304f71bb412fR93,
so lets try reducing this to 0.5 second and let user fall back to
sendText. We may need further investigate why
onDidChangeTerminalShellIntegration may be taking awhile.
remove the need for the EOT in the communication design between the
extension and the python subprocesses produced to run testing.
…4250)

Resolves: #24122
Related: #24190,
#24127

I think the culprit was we were not passing in interpreter when we call
createActivatedEnvironment.
@karthiknadig karthiknadig added the no-changelog No news entry required label Oct 7, 2024
@karthiknadig karthiknadig self-assigned this Oct 7, 2024
@karthiknadig karthiknadig marked this pull request as ready for review October 7, 2024 20:11
@karthiknadig karthiknadig merged commit 2308648 into release/2024.16 Oct 7, 2024
72 checks passed
@karthiknadig karthiknadig deleted the cherry1 branch October 7, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants