Skip to content

Build backend with docker#527

Open
phlmn wants to merge 9 commits into
mainfrom
web-docker
Open

Build backend with docker#527
phlmn wants to merge 9 commits into
mainfrom
web-docker

Conversation

@phlmn
Copy link
Copy Markdown
Member

@phlmn phlmn commented May 16, 2026

No description provided.

@phlmn phlmn force-pushed the web-docker branch 11 times, most recently from d88e2d8 to 0a0208a Compare May 16, 2026 16:14
@phlmn phlmn changed the title Build docker image for backend Build backend using docker May 16, 2026
@phlmn phlmn changed the title Build backend using docker Build backend with docker May 16, 2026
@phlmn phlmn marked this pull request as ready for review May 16, 2026 16:18
@phlmn phlmn force-pushed the web-docker branch 2 times, most recently from 6cff2d5 to 458a0ea Compare May 16, 2026 16:21
@phlmn phlmn requested a review from anuejn May 16, 2026 16:22
Copy link
Copy Markdown
Member

@anuejn anuejn left a comment

Choose a reason for hiding this comment

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

could you also delete the nix folder?

Comment thread .github/workflows/publish-docker.yml
Comment thread packaging/setup_backend.sh Outdated
Comment thread compose-dev.yml Outdated
@phlmn phlmn force-pushed the web-docker branch 4 times, most recently from 6e70690 to 1afa048 Compare May 16, 2026 23:32
@phlmn phlmn requested a review from anuejn May 17, 2026 13:01
Comment thread .python-version
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would like to not have this as a toplevel file. I would be okay to leave the versions separate (we already have to change it in multiple places anyway) and this suggests, that this is the only place to change it (it is not).

Comment thread doc/development_setup.md
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I actually like the dev.sh being in packaging

Comment thread docker-devshell.sh
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

could you move this to packaging/ and document this somewhere?

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