Skip to content

ci: add macOS/Windows PR check workflow - #2

Open
JJs23 wants to merge 3 commits into
mainfrom
ci/pr-check-cross-platform
Open

ci: add macOS/Windows PR check workflow#2
JJs23 wants to merge 3 commits into
mainfrom
ci/pr-check-cross-platform

Conversation

@JJs23

@JJs23 JJs23 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 미러 레포에 macOS/Windows PR 검증 워크플로 신규 추가
  • 기존 pr-check.yml은 Linux 단일 검증 → 본 워크플로가 macOS-latest, windows-latest 매트릭스 빌드 추가
  • 빌드 단계: setup → cargo test → cargo clippy → npm run build (lint/version-sync는 Linux 워크플로에 위임)

미러 단독 파일 처리

  • 본 파일은 upstream(JJs23/racemo)에는 두지 않음
  • sync-public-whitelist.mjsMIRROR_ONLY_PRESERVE 목록에 추가하여 sync 시 보존됨
  • upstream의 sync 스크립트 변경은 별도 커밋으로 처리

Test plan

  • 본 PR 위에서 macOS-latest 잡이 통과
  • 본 PR 위에서 windows-latest 잡이 통과
  • fail-fast: false 동작 확인 (한쪽 실패해도 다른쪽 계속)

기존 pr-check.yml은 Linux만 검증한다. 외부 컨트리뷰터가 본 미러
레포에 PR을 올릴 때 macOS/Windows 빌드 회귀를 함께 잡기 위해
별도 워크플로를 추가한다.

본 파일은 미러 단독 파일이며 upstream(JJs23/racemo)에서는
sync-public-whitelist.mjs의 MIRROR_ONLY_PRESERVE 목록으로
sync 시 보존된다.
@github-actions github-actions Bot added the area:ci CI / workflows label Jun 17, 2026
pty_reader_pauses_without_acks_and_resumes_losslessly 테스트가
GitHub-hosted macos-latest 러너에서 60s 안에 688KB 출력을 끝까지
받지 못해 실패. 로컬 macOS arm64에서는 ~37s로 통과하므로
flow control 로직의 회귀가 아니라 러너 성능 차이.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the area:backend Tauri Rust backend label Jun 17, 2026
1) src/auth.rs:246의 log::warn! 매크로에서 ?expr syntax (tracing
   crate 전용) 제거. log crate은 미지원이라 Windows 빌드가 syntax
   error로 실패했다.

2) main에서는 이미 sync-public-whitelist.mjs의 whitelist 갱신으로
   .github/workflows/release-linux.yml이 제거됐다. 본 PR 브랜치에는
   분기 시점의 구버전이 남아 있어 머지 시 main에 다시 들어올 위험이
   있으므로 함께 제거한다.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the area:security Security-sensitive label Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Tauri Rust backend area:ci CI / workflows area:security Security-sensitive

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant