Skip to content

ci: Rust release pipeline과 vendor staging 추가 - #10

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/rust-release-pipeline
Apr 18, 2026
Merged

ci: Rust release pipeline과 vendor staging 추가#10
JeremyDev87 merged 1 commit into
masterfrom
codex/rust-release-pipeline

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

요약

  • CI에 cargo fmt --check, cargo clippy --workspace --all-targets -- -D warnings, cargo test --workspace quality gate를 추가했습니다.
  • release workflow를 확장해 플랫폼별 Rust binary를 build하고 vendor/<triple>/legolas[.exe] layout으로 stage/verify 후 draft release asset으로 업로드하도록 구성했습니다.
  • 로컬 재현용 stage-local-vendor-binary.mjs, verify-vendor-layout.mjs, vendor/README.md를 추가했습니다.
  • quality gate를 실제로 켤 수 있도록 기존 Rust 코드의 clippy/fmt baseline도 함께 정리했습니다.

검증

  • cargo fmt --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • cargo build --release -p legolas-cli
  • node ./scripts/stage-local-vendor-binary.mjs --vendor-dir /tmp/legolas-vendor.biFahz
  • node ./scripts/verify-vendor-layout.mjs --vendor-dir /tmp/legolas-vendor.biFahz aarch64-apple-darwin

@JeremyDev87 JeremyDev87 added the ci label Apr 18, 2026
@JeremyDev87 JeremyDev87 self-assigned this Apr 18, 2026
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Findings: 없음
  • Rationale: 헤드 브랜치 worktree에서 cargo test --workspace, cargo fmt --check, cargo clippy --workspace --all-targets -- -D warnings, cargo build --release -p legolas-cli를 직접 실행했고, 새 vendor staging/verify 스크립트도 실제 바이너리로 통과했습니다. 릴리스/CI 변경에서 병합을 막을 만한 재현 가능한 깨짐은 확인되지 않았습니다.

참고: self-authored PR이라 공식 approve/request-changes review 대신 PR comment로 게시했습니다.

@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 18, 2026 15:33
@JeremyDev87
JeremyDev87 merged commit 3e2266f into master Apr 18, 2026
7 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/rust-release-pipeline branch April 18, 2026 15:33
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