Skip to content

Improve docker instructions - #1652

Merged
white238 merged 6 commits into
developfrom
task/white238/improve_docker_instructions
Aug 27, 2026
Merged

Improve docker instructions#1652
white238 merged 6 commits into
developfrom
task/white238/improve_docker_instructions

Conversation

@white238

Copy link
Copy Markdown
Member

@white238
white238 requested review from btalamini, chapman39 and tupek2 and a lite review from Copilot August 26, 2026 22:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates the Smith developer guide’s Docker environment page to clarify platform-specific usage and align examples with the Smith repo/user conventions.

Changes:

  • Added notes for Apple Silicon Macs (amd64 platform flag) and clarified the Docker Hub namespace (seracllnl).
  • Updated docker run example and in-container paths from serac to smith.
  • Updated the example image tag used in the run command.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/docs/sphinx/dev_guide/docker_env.rst Outdated
Comment thread src/docs/sphinx/dev_guide/docker_env.rst Outdated
Comment thread src/docs/sphinx/dev_guide/docker_env.rst Outdated
@white238 white238 changed the title Task/white238/improve docker instructions Improve docker instructions Aug 26, 2026
@white238
white238 merged commit c23cdf8 into develop Aug 27, 2026
17 checks passed
@white238
white238 deleted the task/white238/improve_docker_instructions branch August 27, 2026 22:10
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.

5 participants