Skip to content
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

frontend-week01-과제 #104

Merged
merged 1 commit into from
Dec 17, 2024
Merged

frontend-week01-과제 #104

merged 1 commit into from
Dec 17, 2024

Conversation

thgml21004
Copy link

frontend-week01-과제

Comment on lines +25 to +26
],
};
Copy link

Choose a reason for hiding this comment

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

파일 끝에 붙어 있는 ⛔️ (No newline at end of file) 를 보신적이 있으신가요??
Files changed 를 유심히 보셨다면 한 번씩은 보셨을텐데요.

항상 Pull Request를 보내고 나서 자신이 작성한 코드를 Files changed 에서 다시 한번 살펴보는 습관을 들이시길 추천해요.

파일 끝에 개행을 추가하지 않으면 파일 끝에 ⛔️ (No newline at end of file) 경고 문구가 붙어요.

파일 끝에 개행을 추가 하는 이유는 예전에는 컴파일러가 파일 끝에 개행문자가 없으면 한 줄이 끝나지 않은 것으로 인식해서 에러가 발생하는 이슈가 있었기 때문이에요.

최근에는 파일 끝에 개행문자가 없어도 컴파일러에서 별다른 문제가 발생하지 않지만 혹시나 모를 잠재적인 에러나 POSIX 에 명세되어 있기 때문에 개인적으로는 파일 끝에 개행문자를 추가 하시길 권장해요.

Comment on lines +4 to +15
function main() {
const element = document.getElementById('root');

if (!element) {
return;
}

const root = ReactDOM.createRoot(element);

root.render(<App />);
}

Copy link

Choose a reason for hiding this comment

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

자바스크립트쪽에서는 2칸 들여쓰기를 많이 사용하는 편이에요!
에디터나 린트도 세티해서 사용하시면 좋을 것 같습니다.

@moonkii
Copy link

moonkii commented Oct 28, 2024

image

README.md 에 과제의 의도에 맞게 잘 정리하신 것 같아요.
프로젝트 세팅을 미리 준비해놓고 필요할 때마다 내가 정리한 것들을 보고 참고해서 하면 도움이 많이 되실거에요.
저는 개인적으로 커스텀한 세팅을 NPM 에 따로 배포해놓고 필요할때 마다 사용하기도 해요.
프로젝트 세팅의 경우 버전 등 업데이트가 자주 되기 때문에 최신을 유지해주는 것도 좋습니다 :)

@bbhye1 bbhye1 merged commit e07bfb3 into megaptera-kr:thgml21004 Dec 17, 2024
1 check failed
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.

3 participants