Skip to content

Conversation

joyceerhl
Copy link

@joyceerhl joyceerhl commented Jun 10, 2021

Point fix for language server support in the Python extension in VS Code Stable 1.57.0. This fix was submitted to main but needed to be included in a hotfix release.

* notebook renamed to notebooks, notebooks symbols moved to workspace namespace

* Drop usage of NotebookCellMetadata ctor
@github-actions github-actions bot requested a review from karrtikr June 10, 2021 19:20
@github-actions github-actions bot requested a review from paulacamargo25 June 10, 2021 19:20
@joyceerhl joyceerhl added the no-changelog No news entry required label Jun 10, 2021
Copy link

@karrtikr karrtikr left a comment

Choose a reason for hiding this comment

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

No news entry?

@joyceerhl
Copy link
Author

@karrtikr was thinking I'd just update the CHANGELOG directly, but I can also do the news entry and then your team can put that into the CHANGELOG in a separate PR? Whichever y'all prefer

…ow.activeNotebookEditor` (#16381)

* vscode.window.activeNotebookEditor

* window.visibleNotebookEditors
@joyceerhl
Copy link
Author

In the interest of green CI for release I'm porting the other two proposed API-impacted test fixes 245aed0 and 4936a76

@karrtikr
Copy link

Oh I didn't notice this is the release branch. It's fine to edit the changelog then.

Copy link
Member

@IanMatthewHuff IanMatthewHuff left a comment

Choose a reason for hiding this comment

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

Looks good. Tested on 1.57 stable?

@joyceerhl
Copy link
Author

joyceerhl commented Jun 10, 2021

@IanMatthewHuff yes:

image

The build I have is from a CI run on this PR (number is different than any of the ones that have been published to marketplace):

image

@joyceerhl
Copy link
Author

There's a venv test failure on this PR, I'm trying to identify whether it's related to these changes or also a problem on the main branch.

@joyceerhl
Copy link
Author

The test failure (Windows venv) is also present on main. @karthiknadig can this PR be merged?

@joyceerhl joyceerhl merged commit c0a8d13 into microsoft:release-2021.05 Jun 10, 2021
@joyceerhl
Copy link
Author

Merged per conversation with Karthik on Teams.

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.

5 participants