Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test/integration: "Error checking" test is broken #2282

Closed
hyangah opened this issue Jun 8, 2022 · 1 comment
Closed

test/integration: "Error checking" test is broken #2282

hyangah opened this issue Jun 8, 2022 · 1 comment
Assignees
Labels
chore/testing issues related to bugs in test, CI setup FrozenDueToAge Soon

Comments

@hyangah
Copy link
Contributor

hyangah commented Jun 8, 2022

The test started to fail after https://go-review.googlesource.com/c/vscode-go/+/405551 (rearranging diagnostics collection passing) with go1.17 or older versions.

But our kokoro CI failed to catch it because our kokoro CI runs with go1.18 and the test was disabled to workaround while waiting for staticcheck go1.18 support.

(what is worse is we aren't monitoring github long tests workflow failures)

@hyangah hyangah added chore/testing issues related to bugs in test, CI setup Soon labels Jun 8, 2022
@gopherbot gopherbot added this to the Untriaged milestone Jun 8, 2022
@hyangah hyangah self-assigned this Jun 8, 2022
@gopherbot
Copy link
Collaborator

Change https://go.dev/cl/411104 mentions this issue: test/integration: reenable staticcheck and fix "Error checking" test

@hyangah hyangah modified the milestones: Untriaged, vscode-go/later Jun 9, 2022
@golang golang locked and limited conversation to collaborators Jun 9, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
chore/testing issues related to bugs in test, CI setup FrozenDueToAge Soon
Projects
None yet
Development

No branches or pull requests

2 participants