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

경북대 FE_이정민 5주차 Step1 #51

Open
wants to merge 8 commits into
base: userjmmm
Choose a base branch
from

Conversation

userjmmm
Copy link

@userjmmm userjmmm commented Jul 26, 2024

안녕하세요 멘토님, 경북대 FE 이정민입니다.
개인 사정으로 3일 동안 결석을 해서 목요일부터 과제를 진행했으나 완벽히 테스트 코드를 구현할 수 없었습니다.
금요일까지 Step2까지 제출해야 되는 만큼, 참고 자료들과 chatGPT를 활용해서 만든 코드를 제출하게 되었습니다.

비록 테스트가 온전히 돌아가진 않고, 코드도 이상하지만 주말 동안 보충 공부와 멘토님의 리뷰를 바탕으로 잘못된 점을 파악해서 추가적으로 보완하겠습니다.
이번 주도 잘 부탁드리겠습니다. 감사합니다!

Copy link

@sjoleee sjoleee left a comment

Choose a reason for hiding this comment

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

안녕하세요~

지난 번에도 비슷한 리뷰를 남겼는데, 코드 내에서 사용된 의존성이 package.json에서 찾아볼 수 없네요.
이런 상황이 일반적인 개발에서는 만나볼 수 없는 상황입니다.
애초에 node_modules에 패키지가 없으니 필요한 곳에서 참조할 수가 없어 개발이 안되거든요.
확인하셔서 다시 작업하시면 좋겠습니다~

참고로 현재 테스트가 대부분 통과하지 못하고 있네요.

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