Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add docker-compose for running locally #1506

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

NyanKiyoshi
Copy link
Member

This adds a Dockerfile and docker-compose.yaml in order to allow containerized local development.

Warning

It may be unstable, NPM often complains about package-lock.json file being outdated in main branch, something or someone may be pushing broken package-lock.json files which may prevent the docker-compose stack from running (using npm ci)

This adds a Dockerfile and docker-compose.yaml in order to allow containerized local development.
@NyanKiyoshi NyanKiyoshi added the enhancement New feature or request label Mar 20, 2025
@NyanKiyoshi NyanKiyoshi requested a review from a team March 20, 2025 14:44
@NyanKiyoshi NyanKiyoshi self-assigned this Mar 20, 2025
Copy link

vercel bot commented Mar 20, 2025

@NyanKiyoshi is attempting to deploy a commit to the Saleor Commerce Team on Vercel.

A member of the Team first needs to authorize it.

@@ -1,4 +1,5 @@
{
"name": "saleor-docs",
Copy link
Member Author

Choose a reason for hiding this comment

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

Needed otherwise the package-lock.json file will change the application name from saleor-docs to app as NPM was trying to guess the name of the project

@NyanKiyoshi
Copy link
Member Author

After some internal feedback, we will changing the pull request to add support for devcontainers instead

@NyanKiyoshi NyanKiyoshi marked this pull request as draft March 20, 2025 16:17
@NyanKiyoshi NyanKiyoshi removed the request for review from a team March 20, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant