Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

ci: harness-validate.yml 트리거 중복 (on: push + pull_request) 정리 #62

Description

@Ps-Neko

배경

PR #61 (chore/ci-paths-filter) 의 /review 단계에서 발견된 INFORMATIONAL finding. T6 paths-filter 스코프 밖이라 별도 issue/PR 로 분리.

문제

.github/workflows/harness-validate.yml:3-5 의 트리거가 양쪽 활성화:

on:
  push:
  pull_request:

PR push 시 push 이벤트 + pull_request 이벤트 양쪽이 트리거되어 잡이 2회 실행. PR #61 CI 결과에서도 모든 잡 (changes, security, validate-*, validate-all 등) 이 2회 노출되어 확인됨.

Fix 안

on:
  push:
    branches: [main]   # main 직접 push 만 (release commit, hotfix 등)
  pull_request:        # PR 은 PR 이벤트만

영향

  • CI 분 소비량 약 50% 절감 (현재 잡당 2회 → 1회)
  • PR check 결과 중복 제거 → PR UI 가독성 향상
  • main 직접 push (release/tag/hotfix) 는 여전히 검증됨
  • feature branch push 는 PR 생성 후 pull_request 이벤트로만 검증

우선순위

P3 — 기능적 문제 없음. 비용/UX 개선.

참고

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions