Skip to content

Conversation

@heejaechang
Copy link

No description provided.

@heejaechang heejaechang requested a review from kimadeline May 12, 2022 16:46
@heejaechang heejaechang added no-changelog No news entry required skip tests Updates to tests unnecessary labels May 12, 2022
@heejaechang
Copy link
Author

@karrtikr @kimadeline I confirmed this fix the race. once this is in, can we release point release?

@kimadeline kimadeline assigned kimadeline and unassigned karrtikr May 12, 2022
Copy link

@kimadeline kimadeline left a comment

Choose a reason for hiding this comment

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

@karthiknadig can we do another point-release with this fix?

@karthiknadig
Copy link
Member

Certainly, can we add a news item to this.

@heejaechang
Copy link
Author

@karthiknadig do we have issue number that I can use for news entry?

@karthiknadig
Copy link
Member

You can use this #19139

@heejaechang heejaechang removed the no-changelog No news entry required label May 12, 2022
@heejaechang
Copy link
Author

@karthiknadig karthiknadig merged commit 84b85cb into microsoft:main May 12, 2022
karthiknadig pushed a commit to karthiknadig/vscode-python that referenced this pull request May 12, 2022
* moved lsp to latest version that has race fixed * updated middleware * added news entry
karthiknadig added a commit that referenced this pull request May 12, 2022
* Indicate the Python output channel has to be manually filled in the issue template (#19130) * Moved LSP to the latest version that has race fixed (#19136) * moved lsp to latest version that has race fixed * updated middleware * added news entry * Fix news entry encoding. * Update versions and changelog Co-authored-by: Kartik Raj <karraj@microsoft.com> Co-authored-by: Heejae Chang <1333179+heejaechang@users.noreply.github.com>
wesm pushed a commit to posit-dev/positron that referenced this pull request Mar 28, 2024
…-python#19136) * moved lsp to latest version that has race fixed * updated middleware * added news entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip tests Updates to tests unnecessary

4 participants