Disable team management from v7 on - #1257
Merged
Merged
Conversation
Contributor
Author
|
This change is part of the following stack: Change managed by git-spice. |
fkling
force-pushed
the
fkling/remove-teams-management
branch
from
February 19, 2026 11:45
c4889ca to
a9a977d
Compare
eseliger
approved these changes
Feb 19, 2026
fkling
force-pushed
the
fkling/remove-teams-management
branch
from
February 19, 2026 13:29
a9a977d to
03aa243
Compare
This was referenced Sep 9, 2026
marcleblanc2
added a commit
that referenced
this pull request
Sep 28, 2026
Teams were removed in Sourcegraph 7.0. Since #1257 (src-cli 7.0.0) the command has refused to run against 7.x instances, so it only works against instances that are no longer supported. Remove it and its reference page. Drops the "teams.md" entry from the doc golden list and the legacyGroupsWithoutSubcommandPages exemption added in #1375, so every remaining legacy group is now expanded into subcommand pages. Part of https://linear.app/sourcegraph/issue/FE-502 Amp-Thread-ID: https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1 Co-authored-by: Amp <amp@ampcode.com>
marcleblanc2
added a commit
that referenced
this pull request
Sep 28, 2026
* remove: `src teams` commands Teams were removed in Sourcegraph 7.0. Since #1257 (src-cli 7.0.0) the command has refused to run against 7.x instances, so it only works against instances that are no longer supported. Remove it and its reference page. Drops the "teams.md" entry from the doc golden list and the legacyGroupsWithoutSubcommandPages exemption added in #1375, so every remaining legacy group is now expanded into subcommand pages. Part of https://linear.app/sourcegraph/issue/FE-502 Amp-Thread-ID: https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1 Co-authored-by: Amp <amp@ampcode.com> * remove Sourcegraph70 feature flag and stale extensions/gateway help entries Sourcegraph70 was only used by the teams version gate. The `extensions` and `gateway` commands were removed in #1173 and earlier but were still listed in `src help`. Part of https://linear.app/sourcegraph/issue/FE-502 Amp-Thread-ID: https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1 Co-authored-by: Amp <amp@ampcode.com> --------- Co-authored-by: Amp <amp@ampcode.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR makes a version check to disable the teams command for v7 and above.
I refactored the code a bit to make some of the functionality used for batch changes more globally available. No idea if that makes sense.