Skip to content

Fix docker compose#926

Merged
jeswr merged 2 commits intomainfrom
fix-docker-compose
Aug 28, 2025
Merged

Fix docker compose#926
jeswr merged 2 commits intomainfrom
fix-docker-compose

Conversation

@matthieubosquet
Copy link
Member

The structure of the docker-compose is invalid.

This fixes it and allows easily running the website in a Docker container.

Dev workflow:

~/GitHub/solid ❯ git clone git@github.com:solid/solidproject.org.git
~/GitHub/solid ❯ cd solidproject.org/
~/GitHub/solid/solidproject.org ❯ docker compose up

The website gets built in _site/ and is visible at localhost:4000; jekyll running in the container watches the directory and rebuilds on changes.

@netlify
Copy link

netlify bot commented Aug 27, 2025

Deploy Preview for musical-sawine-26ffa9 ready!

Name Link
🔨 Latest commit 6d089ea
🔍 Latest deploy log https://app.netlify.com/projects/musical-sawine-26ffa9/deploys/68b0755c1842300008a3d144
😎 Deploy Preview https://deploy-preview-926--musical-sawine-26ffa9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Vinnl Vinnl requested a review from jeswr August 28, 2025 10:20
Copy link
Member

@jeswr jeswr left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for assinging @Vinnl

I've just created an @solid/odi-team team so we can assign anyone on the ODI team for review going forward!

@matthieubosquet matthieubosquet changed the title Fix docker compose Fix: docker compose Aug 28, 2025
@matthieubosquet matthieubosquet changed the title Fix: docker compose Fix docker compose Aug 28, 2025
@jeswr jeswr disabled auto-merge August 28, 2025 15:27
@jeswr jeswr merged commit 87f44c5 into main Aug 28, 2025
@jeswr jeswr deleted the fix-docker-compose branch August 28, 2025 15:27
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.

2 participants