Skip to content

Resolve merge conflicts and simplify README#796

Merged
ilopezluna merged 1 commit intomainfrom
readme-fix
Mar 26, 2026
Merged

Resolve merge conflicts and simplify README#796
ilopezluna merged 1 commit intomainfrom
readme-fix

Conversation

@ericcurtin
Copy link
Copy Markdown
Contributor

Remove the detailed dmrlet documentation section that was added during merge conflict resolution. This includes the feature comparison table, extensive usage examples, supported backends list, and architecture diagram. The README now contains only a brief introduction to dmrlet with a minimal usage example to keep the documentation concise and focused on the core docker-model-runner project.

Remove the detailed dmrlet documentation section that was added during
merge conflict resolution. This includes the feature comparison table,
extensive usage examples, supported backends list, and architecture
diagram. The README now contains only a brief introduction to dmrlet
with a minimal usage example to keep the documentation concise and
focused on the core docker-model-runner project.

Signed-off-by: Eric Curtin <eric.curtin@docker.com>
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request cleans up the README.md file by removing git merge conflict markers and significantly streamlining the documentation for dmrlet. Key sections such as the feature comparison table, detailed usage examples (including daemon management, scaling, and logging), supported backends, and architecture diagrams have been removed, leaving a simplified 'Serve a model' example using gemma3. I have no feedback to provide.

@ilopezluna ilopezluna merged commit 3fe0e1d into main Mar 26, 2026
12 checks passed
@ilopezluna ilopezluna deleted the readme-fix branch March 26, 2026 18:57
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