팁과 꿀팁

코드 삭제 체크리스트

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

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

  • CSS.

    예를 들어, 이제 사용되지 않는 .mr-card 클래스가 포함된 Vue 컴포넌트를 삭제했습니다. 그렇다면 .mr-card CSS 규칙 집합은 merge_requests.scss에서 삭제되어야 합니다.

  • Ruby 변수.

    사용되지 않는 Ruby 변수를 삭제하는 것은 중요합니다. 왜냐하면 그 변수들을 계속해서 비용이 많이 드는 코드로 초기화하게 되기 때문입니다.

    예를 들어, 남아 있는 페이지 템플릿에서 @total_count Ruby 변수를 더 이상 사용하지 않는 Haml 템플릿을 삭제했습니다. 그러므로 issues_controller.rb에서 @total_count의 초기화는 불필요한 데이터베이스 호출을 줄이기 위해 삭제되어야 합니다.

  • Ruby 메소드.