Skip to content

[chore] CI 워크플로 5.0.x 브랜치 트리거 추가 및 deprecated actions 업그레이드#40

Open
dasomel wants to merge 2 commits into
eGovFramework:mainfrom
dasomel-eGovFramework:chore/ci-5.0.x-trigger-actions-v4
Open

[chore] CI 워크플로 5.0.x 브랜치 트리거 추가 및 deprecated actions 업그레이드#40
dasomel wants to merge 2 commits into
eGovFramework:mainfrom
dasomel-eGovFramework:chore/ci-5.0.x-trigger-actions-v4

Conversation

@dasomel
Copy link
Copy Markdown
Contributor

@dasomel dasomel commented May 20, 2026

변경 사유

현재 .github/workflows/maven.ymlmain 브랜치만 트리거하고 있어 5.0.x 대상 PR이 CI에서 검증되지 않습니다. 또한 actions/checkout@v3/setup-java@v3는 deprecated, distribution: adopt는 Eclipse Adoptium 개명(temurin)에 따라 경고가 발생합니다.

변경 내용

  • push.branches, pull_request.branches"5.0.x" 추가
  • actions/checkout@v3@v4
  • actions/setup-java@v3@v4
  • distribution: 'adopt''temurin'

영향 범위

  • 빌드/테스트 명령은 동일 (mvn -B package --file pom.xml -Dmaven.test.skip=true)
  • list artifact 스텝도 변경 없음
  • 머지 후 5.0.x 대상 push/PR이 CI에서 자동 빌드됨
  • 테스트 실행 활성화는 별도 PR로 분리 예정

체크리스트

  • 단일 주제만 다룸 (CI 인프라 갱신)
  • 5.0.x 브랜치 대상
  • 빌드 명령/스크립트 미변경

- Add "5.0.x" to push and pull_request branch filters so PRs against
  the active 5.x development branch are built.
- Upgrade actions/checkout v3 to v4 (v3 reached end-of-support).
- Upgrade actions/setup-java v3 to v4.
- Replace deprecated distribution "adopt" with "temurin"
  (Eclipse Adoptium renamed; "adopt" emits deprecation warnings).
@eGovFrameSupport
Copy link
Copy Markdown
Contributor

컨트리뷰션 PR은 버전 브랜치가 아닌 main 대상으로 추가 바랍니다.

@dasomel dasomel changed the base branch from 5.0.x to main May 26, 2026 15:53
@dasomel dasomel changed the title [chore][5.0.x] CI 워크플로 5.0.x 브랜치 트리거 추가 및 deprecated actions 업그레이드 [chore] CI 워크플로 5.0.x 브랜치 트리거 추가 및 deprecated actions 업그레이드 May 26, 2026
@eGovFrameSupport
Copy link
Copy Markdown
Contributor

표준프레임워크에 대한 지속적인 참여에 대단히 감사드립니다.
PR을 main으로 재타게팅해 주셔서 정책 정렬됐고, 변경 사항을 다음과 같이 분리해 다뤘으면 합니다.

1. 수용 : Actions 버전 / distribution 갱신

  • actions/checkout@v3@v4
  • actions/setup-java@v3@v4
  • distribution: 'adopt''temurin' (Eclipse Adoptium 명칭 변경, adopt는 deprecation warning 발생)

2. 반려 : branches 필터에 "5.0.x" 추가

이 변경은 GitHub Actions 트리거 사양상 의도한 효과가 발생하지 않습니다.

GitHub Actions의 워크플로 파일 참조 규칙:

  • push 이벤트 → 푸시된 브랜치의 커밋에 있는 워크플로 파일을 읽음
  • pull_request 이벤트 → base(target) 브랜치에 있는 워크플로 파일을 읽음

따라서 main의 maven.yml["main", "5.0.x"]를 적어 두어도

  • 누군가 5.0.x로 push → GA는 5.0.x:.github/workflows/maven.yml을 참조 → 그 파일의 필터는 여전히 ["main"]
    트리거 안 됨
  • 누군가 base가 5.0.x인 PR을 열면 → GA는 5.0.x 브랜치의 워크플로 파일을 참조 → 동일 결과

즉 main의 필터에 "5.0.x"를 추가해도 5.0.x 관련 이벤트는 한 건도 잡지 못합니다. dead config가 되므로 제거하는 것이 맞다고 판단됩니다.

3. 정책 : 버전 브랜치(5.0.x, 4.x.x 등) 동결

본 레포의 운영 원칙: main 외 버전 브랜치는 아카이브이며, 중대한 보안 취약점 핫픽스 외 일반 컨트리뷰션 머지를 받지 않습니다.

@dasomel
Copy link
Copy Markdown
Contributor Author

dasomel commented May 29, 2026

리뷰 반영하여 branches 배열에서 "5.0.x" 필터를 제거했습니다.

현재 이 리포지토리의 기본 브랜치는 main이며, 5.0.x 브랜치는 존재하지 않아 해당 트리거 조건이 실제로 동작하지 않는 dead config였습니다. push/pull_request 트리거 모두 main 단일 브랜치로 정리했습니다. actions/checkout@v4, setup-java@v4, distribution temurin은 그대로 유지합니다.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants