Skip to content

CI에서 baseline 파일이 없어도 dependencyGuard/manifestShield 검사가 통과됨 #337

Description

@fornewid

배경

  • PR Add manifest-shield #336 리뷰에서 발견됨
  • manifestShield와 dependencyGuard는 baseline 파일이 없으면 새로 만들고 성공으로 처리합니다.
  • 예를 들어 flavor를 추가해 configuration("prodRelease")로 바꾼 뒤 새 baseline을 커밋하지 않으면, CI가 매번 baseline을 새로 만들고 통과합니다. 이 경우 검사가 사실상 꺼진 상태가 됩니다.

해야 할 일

  • CI의 guard step 뒤에 baseline이 커밋되어 있는지 확인하는 step 추가를 검토합니다.
    - name: Verify baselines are committed
      run: test -z "$(git status --porcelain -- app/manifestShield app/dependencies)"

참고

  • 심각도: low
  • 발견 출처: Claude Review Agent

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions