Skip to content
This repository was archived by the owner on Sep 26, 2025. It is now read-only.

Conversation

@dbarrosop
Copy link
Member

@dbarrosop dbarrosop commented Sep 17, 2025

PR Type

Bug fix


Description

  • Replace Hasura GraphQL Engine image with Nhost’s

File Walkthrough

Relevant files
Bug fix
docker.go
Updated GraphQL Engine Docker image source                             

dockercompose/docker.go

  • Changed Docker image reference from hasura/graphql-engine to
    nhost/graphql-engine
  • Preserved version tag cli-migrations-v3
+1/-1     

@github-actions github-actions bot added the fix label Sep 17, 2025
@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Image Repository

Verify that the nhost/graphql-engine image exists under the specified path and tag to prevent deployment failures.

fmt.Sprintf("nhost/graphql-engine:%s.cli-migrations-v3", hasuraVersion),
Variable Naming

The hasuraVersion variable is now used for Nhost image tagging; consider renaming it to nhostVersion or similar for clarity.

fmt.Sprintf("nhost/graphql-engine:%s.cli-migrations-v3", hasuraVersion),

@github-actions
Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@dbarrosop dbarrosop merged commit 01e6782 into main Sep 17, 2025
10 checks passed
@dbarrosop dbarrosop deleted the nhost-image branch September 17, 2025 09:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants