docs: Clarify docker instructions to work with spaces in paths#914
docs: Clarify docker instructions to work with spaces in paths#914MaxymVlasov merged 2 commits intomasterfrom
Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe README file was updated to modify the Docker run command example. The volume mount argument for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changesNo out-of-scope changes found. 📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Pull Request Overview
This PR addresses issue #913 by adding quotes around the volume mount path in Docker commands to handle directory paths containing spaces. This ensures the Docker command works correctly regardless of the workspace location.
- Added quotes around
$(pwd):/lintin the Docker volume mount parameter
Co-authored-by: George Yermulnik (Georgii Iermulnik) <yz@yz.kiev.ua>
7f1408b to
289e4f4
Compare
|
This PR is included in version 1.100.0 🎉 |
Description of your changes
Fixes #913