Skip to content

[CHORE#86] CI ruff 버전을 uv.lock 과 고정, 0.17 포맷 드리프트 정리 - #87

Open
juhy0987 wants to merge 1 commit into
mainfrom
chore/#86/ruff-format-pin
Open

juhy0987 wants to merge 1 commit into
mainfrom
chore/#86/ruff-format-pin

Conversation

@juhy0987

Copy link
Copy Markdown
Member

연관 이슈


구현 내용

  • ci-quality.yml Format Check 의 pip install ruff (최신) 를 pip install "ruff==0.17.0" 으로 고정 — uv.lock 의 ruff 버전과 동일. 로컬 uv run ruff 와 CI 가 같은 버전으로 판정하도록 한다. ruff 를 올릴 때는 uv.lock · 이 줄 · 포맷 결과를 같은 PR 에서 함께 바꾼다는 주석 추가
  • ruff 0.17 은 Markdown 파일 안의 Python 코드 블록도 포맷 검사한다. 지적된 파일 4개 (.claude/rules/ 규칙 문서의 코드 블록) 를 ruff format 으로 재포맷. 코드 변경 없음
  • 로컬 검증: uvx ruff@0.17.0 format --check . → 152 files already formatted, uvx ruff@0.17.0 check . → All checks passed

결정 (ADR — 하네스 07-decisions.md 우선순위 3: PR 본문)

제목: CI 의 ruff 를 uv.lock 버전으로 고정 · 날짜: 2026-10-11 · 유형: 운영 정책 · 상태: 채택 · 결정자: AI 자율 (규약 내)

  • 맥락: PR [DOCS#84] 공통 하네스(~/.claude) 참조로 AI 작업 규약 재구성 #85 (문서만 변경) 의 Format Check 가 PR 과 무관한 파일로 실패. CI 가 최신 ruff 를 설치해 로컬과 다른 버전으로 판정했다.
  • 결정: CI 설치 버전을 uv.lock 과 같은 0.17.0 으로 고정하고, 0.17 기준으로 문서의 코드 블록을 재포맷.
  • 검토한 대안: (a) CI 를 uv run ruff 로 전환 — setup-uv 도입과 캐시 설정이 필요해 범위가 커진다. 다음 CI 정비 때 검토. (b) Markdown 을 ruff format 대상에서 제외 — 문서 코드 블록의 포맷 일관성을 포기한다. 기각.
  • 결과 · 영향: 로컬 .venv 가 lock 보다 오래된 ruff 를 갖고 있으면 make lint 결과가 CI 와 다를 수 있다 — uv sync 로 맞춘다.
  • 되돌리기: 이 PR revert.

작업 분배 (서브 에이전트 사용 시)

하위 작업 역할 티어 검수 결과
없음 — — —

CI / 머지 게이트 점검

CI 운영 규약 및 Required Status Checks 단일 소스에 따라 작성합니다.

변경 영향 범위

  • 영향 패키지/모듈: .github/workflows/ci-quality.yml, .claude/rules/*.md (코드 블록 포맷만) — Python 코드 변경 없음
  • 위험도(택1): Low

Required Status Checks

  • 통과 확인 대상 (PR Checks 탭에서 확인):
    • Commit Lint — [CHORE]: 1건
    • PR Title Lint
    • Linked Issue Check — Closes #86
    • Format Check
    • Build
    • Test
    • Lint

롤백 계획

  • git revert 1 커밋.

TODO


논의 사항

  • CI 를 uv run ruff 로 전환해 lock 과 자동으로 맞추는 안은 별도 [CHORE] 로.

🤖 Generated with Claude Code

- ci-quality Format Check 가 최신 ruff 를 설치해 로컬(uv.lock)과 판정이 어긋났음
- ruff 0.17 은 Markdown 안의 Python 코드 블록도 포맷 검사 — 규칙 문서 코드 블록 재포맷

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 11, 2026 09:09
@juhy0987 juhy0987 added the chore 문서, 의존성 등 코드 외 부가 작업 label Oct 11, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Lint job은 여전히 최신 ruff를 설치하므로 CI와 로컬의 버전 불일치가 완전히 해소되지 않습니다.

1 open finding
What changed in this PR

CI Format Check의 ruff를 로컬 버전과 맞추고 Markdown 코드 블록의 포맷 드리프트를 정리합니다.

Changes:

  • Format Check에서 ruff 0.17.0 사용
  • 규칙 문서의 Python 코드 블록 재포맷
  • 버전 갱신 절차를 워크플로 주석으로 명시
File Description
.github/​workflows/​ci-quality.yml Format Check의 ruff 버전 고정
.github/​copilot-instructions.md Python 예제 포맷 정리
.claude/​rules/​06-code-style.md 코드 스타일 예제 포맷 정리
.claude/​rules/​05-testing.md 테스트 예제 포맷 정리
.claude/​rules/​04-error-handling.md 예외·로깅 예제 포맷 정리

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

run: pip install ruff
# uv.lock 의 ruff 버전과 맞춘다 — 버전이 다르면 로컬과 CI 의 포맷 판정이 어긋난다 (이슈 #86).
# ruff 를 올릴 때는 uv.lock · 이 줄 · 포맷 결과를 같은 PR 에서 함께 바꾼다.
run: pip install "ruff==0.17.0"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 문서, 의존성 등 코드 외 부가 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE] CI Format Check 가 main 에서 ruff format 드리프트로 실패

2 participants