Skip to content

Setup and publish a docker image for use in ethui-stacks#255

Merged
naps62 merged 11 commits intomainfrom
docker
Oct 20, 2025
Merged

Setup and publish a docker image for use in ethui-stacks#255
naps62 merged 11 commits intomainfrom
docker

Conversation

@naps62
Copy link
Copy Markdown
Member

@naps62 naps62 commented Oct 15, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 15, 2025 18:34
@naps62 naps62 requested a review from joaocosta9 as a code owner October 15, 2025 18:34
@vercel
Copy link
Copy Markdown

vercel bot commented Oct 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ethui-explorer Ready Ready Preview Comment Oct 17, 2025 2:57pm

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR establishes Docker containerization and automated publishing for the ethui-stacks project to enable deployment and distribution via container images.

  • Adds Docker containerization with multi-stage build configuration
  • Implements GitHub Actions workflow for automated Docker image publishing to GitHub Container Registry
  • Updates package dependencies and adds production start script

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
package.json Adds production start script and updates rollup dependency
Dockerfile Creates containerization setup with Node.js 22 and production build
.github/workflows/ci.yml Implements CI workflow for multi-platform Docker image publishing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
jobs:
docker-publish:
runs-on: ubuntu-latest
needs: test
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow references a 'test' job that doesn't exist in this file, which will cause the docker-publish job to fail. Either add the missing test job or remove the 'needs: test' dependency.

Suggested change
needs: test

Copilot uses AI. Check for mistakes.
@naps62 naps62 merged commit d09d777 into main Oct 20, 2025
4 checks passed
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.

3 participants