팁과 요령
코드 삭제 체크리스트
병합 요청에서 코드를 삭제할 때, 더 이상 사용되지 않는 모든 관련 코드를 삭제하는 것이 중요합니다.
Haml 및 Vue 코드를 삭제할 때, 다음과 같은 사용되지 않는 코드 유형이 포함되어 있는지 확인하세요:
-
CSS.
예를 들어, 이제 더 이상 사용되지 않는
.mr-card
클래스를 포함한 Vue 컴포넌트를 삭제했습니다.
그러므로.mr-card
CSS 규칙 세트는merge_requests.scss
에서 삭제해야 합니다. -
루비 변수.
사용되지 않는 루비 변수를 삭제하는 것은 중요합니다.
그렇지 않으면 잠재적으로 비싼 코드로 변수를 계속 인스턴스화하게 됩니다.예를 들어,
@total_count
루비 변수를 사용한 Haml 템플릿을 삭제했습니다.
@total_count
변수는 페이지의 나머지 템플릿에서는 더 이상 사용되지 않았습니다.
문제 카운트를 계산하기 위한 불필요한 데이터베이스 호출을 방지하기 위해issues_controller.rb
에서@total_count
인스턴스화도 삭제해야 합니다. -
루비 메서드.