Skip to content

test(cli): remove Docker manifest alignment check#5488

Merged
jgoux merged 1 commit into
developfrom
jgoux/remove-docker-alignment-test
Jun 5, 2026
Merged

test(cli): remove Docker manifest alignment check#5488
jgoux merged 1 commit into
developfrom
jgoux/remove-docker-alignment-test

Conversation

@jgoux
Copy link
Copy Markdown
Contributor

@jgoux jgoux commented Jun 5, 2026

Removes the unit test that compared the TypeScript services image matrix with the Go Dockerfile manifest.

That assertion caused Docker-only Dependabot PRs to fail CI whenever Dependabot updated the Dockerfile but not the TypeScript mirror. Keeping the check out of the unit suite lets those automated dependency bumps merge when CI is otherwise green.

Also broadens the CLI e2e parity normalizer so two-component Docker tags like v14.13 are treated as volatile version output, matching the existing behavior for three-component tags. This prevents Go/TypeScript parity from failing solely because Dependabot bumped a Docker image tag.

Finally narrows LOCAL_SERVICE_IMAGES to module scope now that tests no longer import it.

@jgoux jgoux requested a review from a team as a code owner June 5, 2026 11:58
@jgoux jgoux force-pushed the jgoux/remove-docker-alignment-test branch from 8bf3394 to def3c67 Compare June 5, 2026 12:04
@jgoux jgoux enabled auto-merge (squash) June 5, 2026 12:05
@jgoux jgoux merged commit 0c6c4c9 into develop Jun 5, 2026
11 checks passed
@jgoux jgoux deleted the jgoux/remove-docker-alignment-test branch June 5, 2026 12:09
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

Supabase CLI preview

npx --yes https://pkg.pr.new/supabase@5488

Preview package for commit def3c67.

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