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

Songwoojeong PR #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Songwoojeong PR #3

wants to merge 4 commits into from

Conversation

SongWooJeong91
Copy link

송우정 미니 쇼핑몰 pull request

src/main.js Outdated
// 이벤트 리스너를 이용해 버튼 필터링
setEventListeners(items);
})
.catch(console.log);
Copy link
Contributor

Choose a reason for hiding this comment

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

지금 마지막에 ⛔ 이러한 이모티콘이 있는 이유는 개행을 하지 않아서인데요,
파일 끝에는 항상 개행을 해야 한다고 하네요!
그렇지 않으면 이 파일이 아직 끝나지 않았다고 인식을 하기 때문인데요,
이거를 EOL(End of Line)이라고 표현하더라고요.
자세한 내용은 참고 링크를 보시고 이해하시면 될 거 같습니다!

참고 링크: https://velog.io/@doondoony/posix-eol
p.s 'prettier' 확장 프로그램을 사용하시면 자동으로 EOL을 해주기 때문에, 'prettier' 사용을 추천드려요~!

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.

2 participants