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

request: please keep the engine at 1.58 #1688

Closed
OneOfOne opened this issue Aug 13, 2021 · 8 comments
Closed

request: please keep the engine at 1.58 #1688

OneOfOne opened this issue Aug 13, 2021 · 8 comments

Comments

@OneOfOne
Copy link
Contributor

This is related to microsoft/vscode#130457

A lot of us are hardlocking vscode to v1.58.x until we find an alternative, so would it be possible to use the 1.58 engine in package.json until there's a strong reason to switch to 1.59?

@gopherbot gopherbot added this to the Untriaged milestone Aug 13, 2021
@hyangah
Copy link
Contributor

hyangah commented Aug 13, 2021

Thanks for the info @OneOfOne
I thought VS Code users still could use v0.27.1 and marketplace would suggest only compatible versions of the extension. Is it not the case?

We wanted 1.59 to use the new test explorer API in the next release. @firelizzard18 Is it possible to have the extension still require 1.58, and enable the test explorer feature (and use of the new API) only if the api is available (i.e. user is using 1.59 or newer)?

UPDATE: I just tested with Go nightly, and it suggested 2021.8.619 version which is the last go nightly built with the ^1.52.0 engine requirement.

@firelizzard18
Copy link
Contributor

@hyangah I added a commit to bump the engine back to 1.58 and hide the test controller registration behind a check. I think that should do it.

@firelizzard18
Copy link
Contributor

firelizzard18 commented Aug 13, 2021

[golang.go]: testing/item/context is a proposed menu identifier and is only available when running out of dev or with the following command line switch: --enable-proposed-api golang.go

The testing menu contribution blocks the extension from loading on 1.58.

@hyangah
Copy link
Contributor

hyangah commented Aug 16, 2021

Thanks @firelizzard18 for investigating and filing the issue upstream.
It's unfortunate but I am voting to switch to 1.59 at least in nightly, and let users of vscode 1.58 keep using v0.27.1 or earlier versions.

@hyangah
Copy link
Contributor

hyangah commented Aug 17, 2021

I will leave this issue open - we will reevaluate this decision during v0.28.0 release.

@hyangah hyangah modified the milestones: Untriaged, v0.28.0, v0.27.1 Aug 17, 2021
@OneOfOne
Copy link
Contributor Author

Understandable, thank you for looking into it.

@firelizzard18
Copy link
Contributor

The VSCode team is unwilling to support the tools necessary for this extension to utilize new features in a backwards compatible way: microsoft/vscode#130796 (comment).

@hyangah
Copy link
Contributor

hyangah commented Sep 16, 2021

Noted the requirement in the change log.
Sorry that the vscode issues didn't go in the direction we wanted. I hope that v0.28.2 does not have critical issues and remains usable for users who can't upgrade.

@hyangah hyangah closed this as completed Sep 16, 2021
@golang golang locked and limited conversation to collaborators Sep 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants