팁과 요령

코드 삭제 체크리스트

병합 요청에서 코드를 삭제할 때, 더 이상 사용되지 않는 모든 관련 코드를 삭제하는 것이 중요합니다.

Haml 및 Vue 코드를 삭제할 때, 다음과 같은 사용되지 않는 코드 유형이 포함되어 있는지 확인하세요:

  • CSS.

    예를 들어, 이제 더 이상 사용되지 않는 .mr-card 클래스를 포함한 Vue 컴포넌트를 삭제했습니다.
    그러므로 .mr-card CSS 규칙 세트는 merge_requests.scss에서 삭제해야 합니다.

  • 루비 변수.

    사용되지 않는 루비 변수를 삭제하는 것은 중요합니다.
    그렇지 않으면 잠재적으로 비싼 코드로 변수를 계속 인스턴스화하게 됩니다.

    예를 들어, @total_count 루비 변수를 사용한 Haml 템플릿을 삭제했습니다.
    @total_count 변수는 페이지의 나머지 템플릿에서는 더 이상 사용되지 않았습니다.
    문제 카운트를 계산하기 위한 불필요한 데이터베이스 호출을 방지하기 위해 issues_controller.rb에서 @total_count 인스턴스화도 삭제해야 합니다.

  • 루비 메서드.