-
Notifications
You must be signed in to change notification settings - Fork 0
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
huijeong-kim님 기여 #10
Comments
지난 번에 본다고 한 #3906 이슈는 다른 분이 보고 계셔서 다른 good first issue중 #3818을 골라 PR 올려 봤습니다..! 이렇게 바로 올리면 되는 건지 모르겠네요 ㅎㅎ |
지난 번에 올린 PR 은 머지되었습니다. 근데 머지 후 ci test에서 fail이 발생해서 revert 되었어요. 해당 테스트를 로컬에서 돌려보는 데 애를 먹었는데(결국 못 돌렸어요), 기존 PR 리뷰어분이 로컬에서는 fail이 나지 않는데 서버에서만 문제가 생긴다고 하더라고요. 리뷰어분이 코드를 다음과 같이 수정해 주시고 다시 머지되었습니다. fail code let nonames = *matches
.try_get_one("no-names")
.unwrap_or(Some(&false))
.unwrap(); pass code let nonames = *matches
.try_get_one("no-names")
.unwrap_or(None)
.unwrap_or(&false); 서버 테스트 fail 을 확인하는 과정이 매우 어려웠습니다 ㅠㅠ 리뷰어분 도움이 없었으면 못했을 것 같아요ㅠㅠ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: