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

Update engine version to match LSP client #24065

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

rchiodo
Copy link

@rchiodo rchiodo commented Sep 6, 2024

No description provided.

@rchiodo rchiodo added the no-changelog No news entry required label Sep 6, 2024
@rchiodo rchiodo requested a review from karthiknadig September 6, 2024 17:32
@rchiodo rchiodo self-assigned this Sep 6, 2024
@rchiodo
Copy link
Author

rchiodo commented Sep 9, 2024

@karthiknadig are you okay with this going in? This would be for the hotfix branch? Not sure where you generate hotfixes from.

@karthiknadig
Copy link
Member

This is fine. But it should probably also be updated n main. Typically we put fix in main then pull it into branch

@rchiodo rchiodo changed the base branch from release/2024.14 to main September 9, 2024 20:11
@rchiodo
Copy link
Author

rchiodo commented Sep 9, 2024

Okay I'll rebase it

@rchiodo rchiodo force-pushed the rchiodo/update_engine_version branch from e952e65 to 3c3616c Compare September 9, 2024 20:18
@vs-code-engineering vs-code-engineering bot added this to the September 2024 milestone Sep 9, 2024
@rchiodo rchiodo merged commit 08e7fdf into microsoft:main Sep 9, 2024
40 checks passed
anthonykim1 pushed a commit to anthonykim1/vscode-python that referenced this pull request Sep 9, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
karthiknadig added a commit to karthiknadig/vscode-python that referenced this pull request Sep 9, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
anthonykim1 pushed a commit to anthonykim1/vscode-python that referenced this pull request Sep 13, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
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.

2 participants