-
Notifications
You must be signed in to change notification settings - Fork 8
fix: Alloy 오류 수정 #305
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Alloy 오류 수정 #305
Conversation
Walkthrough
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🔇 Additional comments (2)
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. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다!
strip_components 방식은 불필요한 디렉토리 구조가 없을 거 같다는 생각도 드는데
그냥 docs/ 경로를 유지하고 도커 파일을 수정해 이를 참고
이렇게 하신 이유가 개인적으로 궁금합니다! 지금 방식도 좋습니다!
이후 docs를 재활용할 수 있고, strip_components 방식은 시도해 본적이 없어 실험적이니 좀더 안정적인 쪽을 선택했습니다! |
관련 이슈
작업 내용
alloy.config라는 디렉토리가 알수없는 이유로 생성되어 있어 발생한 문제였습니다.
파일 scp시에 문제가 있었는데
다음과 같이 전송하면 config.alloy가 이동되는게 아닌, docs/config.alloy 전체가 이동되는걸 알지 못했습니다.
strip_components 옵션으로 해결도 가능하나, 그냥 docs/ 경로를 유지하고 도커 파일을 수정해 이를 참고하게 했습니다
특이 사항
리뷰 요구사항 (선택)