Skip to content

VSCE: Remove extension tests from main branch client checks

Created by: tjkandala

We currently run integration tests for our VS Code and browser extensions on each main branch build. However, I do not believe they are necessary to ensure reliability of the extensions. We should stop running them on main branch client checks because:

  • They have nothing to do with Sourcegraph "core" release process
  • Our goal is to not have flakes, but we would have higher flake tolerance on, say, the browser extension release branch that we babysit when releasing the extension vs. on main where other engineers would be blocked by a flake

TODO:

  • Remove VS Code extension smoke test (#34136)
  • Ensure extension tests run only on client PRs, nightly checks, and release branch
    • For browser extension
    • For VS Code extension
  • Don't run web tests on client/vscode only changes (see https://github.com/sourcegraph/sourcegraph/pull/34213#issuecomment-1104231975). Tag FE platform to make sure there aren't any unforeseen consequences. Possibly w/ node modules?