Haze
Haze
그리고 제 생각에는 그... geocoding을 별도의 모듈로 뽑아야할 것 같은데, 좋은 의견을 받습니다.
`OPENWEATHER_API_KEY` 환경변수가 없어서 전체 테스트를 확인하지 못해 코드 커버리지가 낮을 수 있을 것 같습니다. @item4 PR의 코드와 테스트 환경 설정을 한번 확인 부탁드려도 될까요? Test Case와 Type Hint에는 문제가 없어도...
제가 실수로 emcache m1에서 작업되게 한 poetry.lock + pyproject.toml을 같이 넣어버려서, 다시 수정해서 제 레포지토리에 force push 했습니다. 죄송합니다ㅜㅜ 이번에 리뷰하실 때는 실제 동작도 점검해주셨으면 좋겠습니다.
아 사용자 시간대로 맞춰서 변경해주는게 빠졌군요. 이거 한번만 먼저 작동 테스트 부탁드릴게요.
I found a bug in my code about Saving its states. I'll fix this issue.
@faridgh1991 Can you describe your dependencies and its version? (Especially Android's support libraries.)
I want to use this feature. How can I help this to get merged?
Is there any plans for support arm64 architecture for edgedb Docker Image? I feel difficult to use docker with edgedb in Apple Silicon or, Graviton 2. I know already edgedb...
@item4 1. 1번은 고치면 좋을 것 같습니다. 반영하겠습니다. 2. 2번도 고치는게 좋다고 생각합니다. 반영하겠습니다. 3. 3번은 저도 item4님 계신 슬랙에서 보고 넣은건데 기본 이모지가 아니었군요. 당연히 수정하겠습니다. 4. 4번과 5번은...
일단 수정은 했는데, coverage에서 제가 작업하거나 의존하지 않은 부분에 coverage가 일관되게 떨어지게 나오네요. 이 부분만 한번 확인해보시고 테스트를 더 추가하는 방향으로 갈지, 아니면 PR을 머지할지 보시면 좋을 것 같습니다. 이번...