Skip to content

[201411108] Onboarding - #2894

Merged
201411108 merged 6 commits into
DaleStudy:mainfrom
201411108:onboarding
Nov 4, 2026
Merged

201411108 merged 6 commits into
DaleStudy:mainfrom
201411108:onboarding

Conversation

@201411108

@201411108 201411108 commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

답안 제출 문제

작성자 체크 리스트

  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

  • 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
  • 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
  • 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

@dalestudy

dalestudy Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

🤖 AI 리뷰(알고리즘 패턴, 시간/공간 복잡도, 학습 현황) 결과는 Checks 탭에서 볼 수 있어요.

@DaleSeo
DaleSeo self-requested a review November 3, 2026 01:31

@DaleSeo DaleSeo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

온보딩을 축하드립니다!

Comment thread to-lower-case/201411108.py Outdated
Comment on lines +8 to +9
if ord(char) >= 65 and ord(char) <= 90:
result.append(chr(ord(char) + 32))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

65, 90, 32가 같은 숫자가 무엇을 뜻하는지 문자 리터럴로 표현하면 더 읽기가 편했을 것 같아요 😀

Suggested change
if ord(char) >= 65 and ord(char) <= 90:
result.append(chr(ord(char) + 32))
if 'A' <= char <= 'Z':
result.append(chr(ord(char) + ord('a') - ord('A')))

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

리뷰 감사드립니다! python을 오랜만에 쓰는데 많은 것들이 새로 생긴 것 같더라고요.
앞으로 가독성에도 신경쓰면서 작성해보겠습니다!

@201411108
201411108 merged commit 22b7370 into DaleStudy:main Nov 4, 2026
2 checks passed
@201411108
201411108 deleted the onboarding branch November 4, 2026 12:16
@parkhojeong

Copy link
Copy Markdown
Collaborator

@201411108 수고하셨습니다. 검토자 체크리스트 확인 부탁드리고, 직전 PR인 #2893 리뷰 부탁드립니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

3 participants