Skip to content

Dockerfile draft#94

Draft
breml wants to merge 4 commits into
coder:mainfrom
breml:dockerfile-from-release
Draft

Dockerfile draft#94
breml wants to merge 4 commits into
coder:mainfrom
breml:dockerfile-from-release

Conversation

@breml

@breml breml commented Nov 11, 2025

Copy link
Copy Markdown

Relates to: #86

@breml breml mentioned this pull request Nov 11, 2025
@ammario

ammario commented Nov 12, 2025

Copy link
Copy Markdown
Member

Do you mind if I add some commits here to reorganize the change as a sample in the documentation?

@breml

breml commented Nov 12, 2025

Copy link
Copy Markdown
Author

No, not at all. Just go ahead.

ammar-agent added a commit that referenced this pull request Nov 12, 2025
- Add Dockerfile to examples/ directory
- Create comprehensive Docker documentation in docs/advanced/docker.md
- Update SUMMARY.md to include Docker page
- Covers building, running, configuring clients, and rule examples
- Relates to PR #94
- Move Dockerfile from root to examples/ directory
- Add docs/advanced/docker.md with full usage guide
- Update SUMMARY.md to include Docker documentation
- Cover building, running, client configuration, and examples
Instead of duplicating all Docker documentation in README,
add a brief reference pointing to the comprehensive docs.
@breml

breml commented Nov 16, 2025

Copy link
Copy Markdown
Author

@ammario Do you want me to change this PR to "ready" instead of draft? What is the best way to move forward with this in you opinion?

@ammario

ammario commented Nov 16, 2025

Copy link
Copy Markdown
Member

Hey, sorry for disappearing here. I've been busy on another project. I wanted to simplify the markdown docs and create some distinction in the docs between the docker-run feature and this docker server approach. If you wanna complete this please go ahead and ill be around for review.

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