Skip to content

Conversation

Jeehay28
Copy link
Contributor

@Jeehay28 Jeehay28 commented Dec 5, 2024

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@Jeehay28 Jeehay28 requested a review from a team as a code owner December 5, 2024 16:25
@github-actions github-actions bot added the js label Dec 5, 2024
@Jeehay28 Jeehay28 removed their assignment Dec 5, 2024
@Jeehay28
Copy link
Contributor Author

Jeehay28 commented Dec 5, 2024

Assignees 제가 실수로 제거한 것 같습니다. 그런데 Reviewers, Projects는 선택하거나 작성할 수가 없습니다.

@HC-kang
Copy link
Contributor

HC-kang commented Dec 6, 2024

@Jeehay28 님 안녕하세요!
현재 3기 팀에서 조회되지 않는것으로 보아, 아마 팀 초대메일 확인 후 Join하지 않으신 것이 아닐까 싶습니다!
권한부여가 되신다면 프로젝트와 주차 등록이 가능하실것으로 보여요!

@Jeehay28
Copy link
Contributor Author

Jeehay28 commented Dec 6, 2024 via email

@SamTheKorean
Copy link
Contributor

@Jeehay28 안녕하세요 초대를 못받았다고 하셔서 기존에 보낸 초대를 취소하고 다시 발송했습니다! 혹시 확인이 안되시면 스팸함 및 깃허브 알림 확인부탁드립니다!

@Jeehay28 Jeehay28 requested a review from ekgns33 December 7, 2024 03:30
@Jeehay28
Copy link
Contributor Author

Jeehay28 commented Dec 7, 2024

감사합니다. 초대 이메일 받고 수락했습니다.

Copy link
Contributor

@EgonD3V EgonD3V left a comment

Choose a reason for hiding this comment

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

좋은 풀이들 감사합니다. 가능하면 시간복잡도와 공간복잡도 분석도 해보시면 도움이 되실 것 같습니다.

Comment on lines +6 to +7
const reg = /[a-z0-9]/g;
const converted = s.toLowerCase().match(reg);
Copy link
Contributor

Choose a reason for hiding this comment

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

upper case도 정규식에 포함하면 논리의 흐름이 더 좋을 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리뷰 감사합니당~

@DaleSeo
Copy link
Member

DaleSeo commented Dec 9, 2024

@Jeehay28 님, 현재 프로젝트에 주차 설정이 되어 있지 않습니다.
Shot 2024-12-09 at 07 02 02

답안 제출 가이드예제 PR 참고 부탁드립니다.

@Jeehay28 Jeehay28 merged commit b504ef4 into DaleStudy:main Dec 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

5 participants