feat: add Docker examples and setup documentation#14
Open
lemonte wants to merge 12 commits intoUniversal-Commerce-Protocol:mainfrom
Open
feat: add Docker examples and setup documentation#14lemonte wants to merge 12 commits intoUniversal-Commerce-Protocol:mainfrom
lemonte wants to merge 12 commits intoUniversal-Commerce-Protocol:mainfrom
Conversation
5 tasks
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
Contributor
|
Thank you for this contribution! Please take action on the comments. We are evaluating the changes and will provide a formal response within 1-2 weeks. |
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
damaz91
reviewed
Jan 14, 2026
- Introduced separate Docker Compose files for Node.js and Python servers. - Updated README.md to reflect new Docker setup documentation. - Removed the old combined Docker Compose file. - Added .dockerignore files for both Node.js and Python to exclude unnecessary files. - Created Dockerfiles for both servers to streamline the build process.
- Added '*.md' to the .dockerignore file for the Python Docker setup to prevent markdown files from being included in the Docker context.
damaz91
reviewed
Jan 14, 2026
- Corrected the Dockerfile paths in `docker-compose.nodejs.yml` and `docker-compose.python.yml`. - Updated the README files for both Node.js and Python servers to improve clarity and organization of setup instructions. - Removed outdated sections from the README files to streamline the documentation.
damaz91
reviewed
Jan 14, 2026
- Moved Docker Compose files for Node.js and Python servers into their respective directories. - Updated README files to reflect new paths and simplify usage instructions. - Added new Docker Compose files for both Node.js and Python servers to enhance clarity and organization.
damaz91
reviewed
Jan 20, 2026
- Modified .dockerignore to include markdown files. - Updated docker-compose.yml to change build context and Dockerfile path. - Adjusted Dockerfile to copy package files and source from the new directory structure.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds comprehensive Docker support for running UCP reference servers, making it easier for developers to get started without installing Node.js, Python, or their dependencies locally.
Changes include:
New Docker setup (
docker/):docker-compose.nodejs.yml- Standalone Node.js server setupdocker-compose.python.yml- Standalone Python server setupDocumentation updates:
samples/README.mdwith Docker sectionnodejs/README.mdpython/server/README.mdBenefits:
Type of change
Checklist:
Pull Request Title
feat: add Docker examples and setup documentation