Skip to content

Remove deprecated tools/enforce-tags and tools/update-docker-tags scripts - #1249

Merged
keegancsmith merged 1 commit into
mainfrom
rm-enforce-tags
Jul 21, 2025
Merged

keegancsmith merged 1 commit into
mainfrom
rm-enforce-tags

Conversation

@keegancsmith

@keegancsmith keegancsmith commented Jul 21, 2025 •

Copy link
Copy Markdown
Member

This removes the deprecated tools/enforce-tags/main.go and tools/update-docker-tags.sh scripts that are no longer used or functional.

The change includes a safety check to ensure update-docker-tags is not referenced anywhere else in the repository before removal.

Test Plan: n/a

Created by Sourcegraph batch change keegan/rm-enforce-tags.

@keegancsmith
keegancsmith requested a review from a team July 21, 2025 10:19
@keegancsmith
keegancsmith enabled auto-merge (squash) July 21, 2025 10:19
@keegancsmith
keegancsmith merged commit c6e58d4 into main Jul 21, 2025
@keegancsmith
keegancsmith deleted the rm-enforce-tags branch July 21, 2025 11:38
marcleblanc2 added a commit that referenced this pull request Oct 30, 2026
Docker Compose is the only deployment type this repo supports. #1386
removed pure-docker/; this removes what was still left over for
sourcegraph/server and pure-docker:

- nginx/: sourcegraph/server nginx config, unreferenced since 2020
- tools/migrate.sh, tools/migration_vars: sourcegraph/server to Docker
  Compose DB migration; the single-container image was removed in 7.0
- tools/tools.go: pinned update-docker-tags, whose script was removed
  in #1249; go mod tidy drops it and dev/ci/images
- renovate.json rule for a Go module path that no longer exists
- .gitignore entries for nginx certs
- db-only-migrate.docker-compose.yaml header: it is for backup/restore,
  not the server migration

Amp-Thread-ID: https://ampcode.com/threads/T-01a18a77-bb16-74a4-b42c-bdbad59a3194
Co-authored-by: Amp <amp@ampcode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants