fix: stdin is not a tty error in husky

[lumirlumir]

안녕하세요😊

Windows 환경에서 yarn을 통해 husky를 사용할 때, stdin is not a tty 에러가 발생하는 이슈가 있어 해당 버그를 수정합니다.

husky 공식 문서 및 issue에 해결 방법이 있어 그대로 적용하였습니다.

• https://typicode.github.io/husky/troubleshoot.html#yarn-on-windows

• v5 stdin is not a tty typicode/husky#850

Progress

• 번역 초안 작성 (Draft translation)
• 공통 스타일 가이드 확인 (Check the common style guide)
• 모범사례 확인 (Check best practices)
• 용어 확인 (Check the term)
• 맞춤법 검사 (Spelling check)
• 리뷰 반영 (Resolve reviews)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
ko-react-dev	🔄 Building (Inspect)	Visit Preview	💬 Add feedback	Sep 23, 2024 6:50am

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
ko-legacy-reactjs-org	⬜️ Ignored (Inspect)	Visit Preview		Sep 23, 2024 6:50am

[github-actions]

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

[hg-pyun]

LGTM

[highspirit7]

@lumirlumir 어디서 이 얘기를 드려야 할지 몰라서 여기에 남깁니다. 저는 mac os를 사용하고 있고, 오늘 이 레포지토리 포크, 클론 후 커밋을 남기려고 할때 아래와 같은 에러가 발생해서 커밋을 할 수가 없었습니다. 제가 확인한 바로는 이 풀리퀘스트로 추가된 common.sh 파일이 CRLF (Carriage Return + Line Feed) 개행문자를 사용하고 있어서 맥에서 실행하는데 생긴 문제였습니다. mac os는 LF(Line Feed) 개행문자를 기반이라서 LF로 변환해주는 도구를 사용한 뒤 커밋을 할 수는 있었습니다. 앞으로 맥이나 리눅스 유저는 이렇게 변환해서 커밋을 해야하는 것인지 어떻게 이 문제를 해결해야하는지 제 수준으로서는 판단할 수가 없어서 일단 이렇게 남겨봅니다.

[lumirlumir]

@lumirlumir 어디서 이 얘기를 드려야 할지 몰라서 여기에 남깁니다. 저는 mac os를 사용하고 있고, 오늘 이 레포지토리 포크, 클론 후 커밋을 남기려고 할때 아래와 같은 에러가 발생해서 커밋을 할 수가 없었습니다. 제가 확인한 바로는 이 풀리퀘스트로 추가된 common.sh 파일이 CRLF (Carriage Return + Line Feed) 개행문자를 사용하고 있어서 맥에서 실행하는데 생긴 문제였습니다. mac os는 LF(Line Feed) 개행문자를 기반이라서 LF로 변환해주는 도구를 사용한 뒤 커밋을 할 수는 있었습니다. 앞으로 맥이나 리눅스 유저는 이렇게 변환해서 커밋을 해야하는 것인지 어떻게 이 문제를 해결해야하는지 제 수준으로서는 판단할 수가 없어서 일단 이렇게 남겨봅니다.

@highspirit7 아 커밋 시 제 실수가 있었네요ㅠㅠ 오류 발견해주셔서 감사합니다🙇‍♂️

CRLF로 인해 발생한 오류가 맞습니다. 제가 Windows 유저다 보니 커밋 시 LF로 변환해서 올려야 하는데 실수로 CRLF로 올려버렸네요🥲🥲

원래 EditorConfig 같은 도구를 이용해서 모든 파일에 LF를 강제하는 것이 일반적입니다. 하지만, 현재 레포지토리에는 그런 툴이 존재하지 않아 문제가 발생했습니다.

해당 내용 수정 보완해서 PR 올리도록 할테니, 나중에 해당 내용 pull 받으시면 문제 없으실겁니다.

혹여, 현재 추가적으로 올릴 커밋 있으시다면 번거로우시더라도 (PR 반영되기 전까지) 잠깐만 직접 LF로 변환해서 사용해주시면 될 것 같습니다.

[highspirit7]

@lumirlumir 빠른 답변 감사합니다! 그런데 LF로 강제되면 윈도우에서 개발하실 때 생기는 문제는 없나요? 제가 윈도우로 개발해본 적이 없어서 질문드려봅니다.

[lumirlumir]

@highspirit7 VScode같은 코드 에디터를 이용하면 LF를 사용하던 CRLF를 사용하던 똑같이 개행을 처리해 주어서 크게 상관이 없습니다👍 그리고 개발할때는 LF가 사실상 표준이라 CRLF는 사용하는 경우가 드뭅니다...ㅠㅠ

[highspirit7]

@lumirlumir 그렇군요. 윈도우에서 어떤 개행으로 되어있던 간에 실행하는데 문제는 없지만, 커밋 시 LF로 변환을 해주는 것이 필요한 거군요. 좋은 정보 알아갑니다. 감사합니다!

[lumirlumir]

@highspirit7 넵👍👍 #1068 PR과 곧 올릴 #1069 PR에 해당 내용 반영할 예정입니다. 나중에 한번 살펴봐 주세요~ 감사합니다.