Skip to content

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

Open
userjmmm wants to merge 8 commits intokakao-tech-campus-2nd-step2:userjmmmfrom
userjmmm:step1-userjmmm
Open

경북대 FE_이정민 5주차 Step1#51
userjmmm wants to merge 8 commits intokakao-tech-campus-2nd-step2:userjmmmfrom
userjmmm:step1-userjmmm

Conversation

@userjmmm
Copy link
Copy Markdown

@userjmmm userjmmm commented Jul 26, 2024

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

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

Copy link
Copy Markdown

@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