Skip to content

fix: release path filter uses push range#94

Merged
tlarbals824 merged 1 commit intodevelopfrom
fix/release-path-filter-push-range
May 5, 2026
Merged

fix: release path filter uses push range#94
tlarbals824 merged 1 commit intodevelopfrom
fix/release-path-filter-push-range

Conversation

@tlarbals824
Copy link
Copy Markdown
Collaborator

@tlarbals824 tlarbals824 commented May 5, 2026

문제

release.ymlpublish-api-client가 PR #93 머지 후 실행되지 않았습니다.

원인은 check-source-changes 단계에서 dorny/paths-filter가 main push에 포함된 변경사항이 아니라, 현재 브랜치 상태 기준의 develop...main 차이를 비교했기 때문입니다.

Changes will be detected between develop and main
git diff refs/remotes/origin/develop...refs/remotes/origin/main

이 경우 develop에 이미 들어가 있던 src/** 변경은 감지되지 않아 src == true가 되지 않았고, 결과적으로 publish-api-client job이 skipped 되었습니다.

수정 내용

paths-filter의 비교 기준을 명시적으로 main push 이벤트의 전후 커밋으로 변경했습니다.

base: ${{ github.event.before }}
ref: ${{ github.sha }}

이제 release workflow는 브랜치 간 차이가 아니라, 실제로 main에 push된 변경 범위를 기준으로 src/**, build.gradle.kts, buildSrc/** 변경 여부를 판단합니다.

기대 효과

  • develop → main release PR 머지 시 포함된 src/** 변경을 정상 감지합니다.
  • 소스 변경이 있는 release에서는 publish-api-client가 실행됩니다.
  • docs-only release에서는 기존처럼 API client publish를 건너뜁니다.

검증

  • git diff --check

Use the push event before/after range so API client publishing is gated by the changes introduced by the main push, not by the current develop/main branch delta.\n\nConstraint: dorny/paths-filter defaults compared develop...main on release pushes, missing src changes already present in develop.\nRejected: Always publish API client | would publish even for docs-only releases.\nConfidence: high\nScope-risk: narrow\nDirective: Keep release path filtering anchored to push event SHAs for main push workflows.\nTested: git diff --check\nNot-tested: GitHub Actions runtime execution
@github-actions github-actions Bot added the bug label May 5, 2026
@tlarbals824 tlarbals824 merged commit 1f19710 into develop May 5, 2026
2 checks passed
@github-actions github-actions Bot deleted the fix/release-path-filter-push-range branch May 5, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant