Skip to content

Conversation

@raa0121
Copy link
Member

@raa0121 raa0121 commented Nov 3, 2025

概要

  • Windows (MSYS) で正しく volume mount できなかった
    • compose を使うと - ".:/path/to/mount" を解釈してくれるため、マルチプラットフォームとして対応できる

Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

LGTM

自分だけだとよくないので、他レビューしてほしい人をアサインしたほうがいいかと思います。

@raa0121 raa0121 requested a review from koron November 26, 2025 12:31
@raa0121
Copy link
Member Author

raa0121 commented Nov 26, 2025

@koron お手すきでレビューお願いします。

- Exclude compose.yaml from output
- Remove a variable "dir" that are no longer needed
@koron
Copy link
Member

koron commented Nov 27, 2025

7091d6f で、指摘した2事項を修正した。

- Changed how to stop the local HTTP server
- Documented how to completely cleanup
@koron koron merged commit 9046a0e into master Nov 27, 2025
@koron koron deleted the fix/docker-compose branch November 27, 2025 01:25
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.

4 participants