Skip to content

add support for running validator in docker#116

Merged
punkle merged 4 commits intomainfrom
docker-support
Jan 16, 2026
Merged

add support for running validator in docker#116
punkle merged 4 commits intomainfrom
docker-support

Conversation

@punkle
Copy link
Contributor

@punkle punkle commented Jan 15, 2026

No description provided.

@punkle
Copy link
Contributor Author

punkle commented Jan 15, 2026

fixes #116

@punkle punkle requested review from a team and Copilot January 15, 2026 16:42
Copy link
Contributor

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 adds Docker support for the backstage-entity-validator, enabling users to run the validator in a containerized environment without needing to install Node.js or dependencies locally.

Changes:

  • Adds a multi-stage Dockerfile for building and running the validator
  • Creates GitHub Actions workflow for automated Docker image publishing to Docker Hub
  • Adds Docker smoke tests to the CI pipeline to verify container functionality
  • Documents Docker usage with examples in the README

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Documents Docker usage with installation instructions and example commands
Dockerfile Implements multi-stage build with node:20-alpine for optimized production image
.github/workflows/docker-publish.yml Automates Docker image publishing on main branch pushes with version tagging
.github/workflows/ci.yml Adds smoke tests validating Docker build and basic container functionality
.dockerignore Excludes unnecessary files from Docker build context

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: actions/checkout@main
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

Using @main for GitHub Actions is not recommended as it can lead to unexpected breaking changes. Pin to a specific version tag (e.g., @v4) to ensure reproducible builds.

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Dockerfile Outdated
@@ -0,0 +1,35 @@
# Build stage
FROM node:20-alpine AS builder
Copy link
Contributor

@Xantier Xantier Jan 16, 2026

Choose a reason for hiding this comment

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

bump to current LTS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@punkle punkle requested a review from Xantier January 16, 2026 08:35
@punkle punkle merged commit 56396ec into main Jan 16, 2026
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