Skip to content

[#679] 해결된 테스트 실패 로그가 잔여 파일로 인해 반복 노출되는 현상을 해결한다#680

Merged
opficdev merged 4 commits into
developfrom
fix/#679-ci-thread
Jul 3, 2026
Merged

[#679] 해결된 테스트 실패 로그가 잔여 파일로 인해 반복 노출되는 현상을 해결한다#680
opficdev merged 4 commits into
developfrom
fix/#679-ci-thread

Conversation

@opficdev

@opficdev opficdev commented Jul 3, 2026

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • CI 실패 원인이 해결된 뒤에도 이전 실패 로그가 PR 스레드에 반복 노출되는 혼선 제거

📝 작업 내용

📌 요약

  • CI 실패 댓글의 테스트 로그 요약 fallback 제거
  • 요약 가능한 실패 진단이 없을 때 명시 문구 표시
  • CI가 성공하면 이전 CI 실패 댓글을 자동 삭제하도록 Report job 정리

🔍 상세

  • 테스트 로그에서 명확한 실패 패턴을 찾지 못한 경우 마지막 25줄을 실패 로그처럼 붙이던 fallback 제거
  • swiftmodulesreport, SwiftLint note 같은 로그 꼬리 정보가 실패 원인처럼 표시되지 않도록 변경
  • 실패 scheme은 유지하되, 간결한 진단을 찾지 못한 경우 artifact 확인 안내 문구 표시
  • build/test가 모두 성공한 경우 기존 ❌ iOS CI failed. bot 댓글을 찾아 삭제하도록 cleanup step 추가
  • 실패한 경우에만 로그 artifact를 다운로드하고 실패 댓글을 작성하도록 step 조건 분리

📸 영상 / 이미지 (Optional)

  • 없음

@opficdev opficdev self-assigned this Jul 3, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request incorrectly changes the import of PresentationShared to PresentationShare in HomeFeature.swift. The review comment correctly identifies that this will cause a compilation error and provides a suggestion to revert the change.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Application/Presentation/HomeTab/Sources/Home/Home/HomeFeature.swift Outdated
@opficdev opficdev merged commit 5321ae4 into develop Jul 3, 2026
6 checks passed
@opficdev opficdev deleted the fix/#679-ci-thread branch July 3, 2026 07:38
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.

해결된 테스트 실패 로그가 잔여 파일로 인해 반복 노출되는 현상을 해결한다

1 participant