Skip to content

Fix: pre-commit 변경#133

Merged
mjy926 merged 1 commit into
developfrom
fix/precommit
Apr 9, 2026
Merged

Fix: pre-commit 변경#133
mjy926 merged 1 commit into
developfrom
fix/precommit

Conversation

@mjy926

@mjy926 mjy926 commented Apr 8, 2026

Copy link
Copy Markdown
Contributor
  • 기존에 사용하고 있던 black, isort, pycln을 삭제했습니다.
  • Pre-commit에서 Ruff를 사용하도록 수정했습니다.

@mjy926 mjy926 self-assigned this Apr 8, 2026
@mjy926 mjy926 requested a review from a team as a code owner April 8, 2026 08:06
@coderabbitai

coderabbitai Bot commented Apr 8, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 4bdd15f4-d2d9-4516-b322-2751b9623258

📥 Commits

Reviewing files that changed from the base of the PR and between 9094234 and f7a7f1b.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (4)
  • .github/workflows/test.yaml
  • .pre-commit-config.yaml
  • .pylintrc
  • pyproject.toml
💤 Files with no reviewable changes (1)
  • .pylintrc

📝 Walkthrough

Walkthrough

린팅 및 포맷팅 도구 모음을 교체합니다. black, pylint, isort, pycln을 제거하고 ruffruff-format으로 대체합니다. 관련 설정 파일과 개발 의존성을 업데이트합니다.

Changes

Cohort / File(s) Summary
린팅 및 포맷팅 도구 설정
.pre-commit-config.yaml, pyproject.toml
black, pylint, isort, pycln 제거하고 astral-sh/ruff-pre-commit 저장소 추가. 새로운 [tool.ruff] 설정 블록 추가 (line-length, lint rules, isort 호환성 포함)
설정 정리
.pylintrc
전체 Pylint 설정 파일 삭제
워크플로우 문서화
.github/workflows/test.yaml
pre-commit 포함 도구 설명 주석 업데이트 ("pylint, black, isort, etc." → "ruff, pyright, etc.")
개발 의존성
pyproject.toml
black, pylint 제거 및 ruff>=0.15.9 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용을 적절히 요약하고 있으며, pre-commit 도구 변경이라는 핵심 변경 사항을 명확히 전달합니다.
Description check ✅ Passed PR 설명이 변경 사항과 밀접한 관련이 있으며, black/isort/pycln 제거와 Ruff 도입이라는 구체적인 내용을 포함하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/precommit

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mjy926 mjy926 merged commit 61603f6 into develop Apr 9, 2026
3 checks passed
@mjy926 mjy926 deleted the fix/precommit branch April 9, 2026 06:52
@coderabbitai coderabbitai Bot mentioned this pull request Apr 23, 2026
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