Skip to content

chore(dev): trim requirements-dev - #307

Merged
soimkim merged 1 commit into
mainfrom
req_dev
Aug 20, 2026
Merged

chore(dev): trim requirements-dev#307
soimkim merged 1 commit into
mainfrom
req_dev

Conversation

@soimkim

@soimkim soimkim commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Simplified the development tooling setup by removing several optional testing, coverage, compatibility, and scanning tools.
    • Retained the core tools for test execution, environment management, and code quality checks.

@soimkim soimkim self-assigned this Aug 20, 2026
@soimkim soimkim added the chore [PR/Issue] Refactoring, maintenance the code label Aug 20, 2026
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f663f089-d018-4132-b0ae-fbd9eea6144f

📥 Commits

Reviewing files that changed from the base of the PR and between dab343d and 1548d40.

📒 Files selected for processing (1)
  • requirements-dev.txt
💤 Files with no reviewable changes (1)
  • requirements-dev.txt

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The development requirements now contain only tox, pytest, and flake8. Five additional development packages were removed.

Changes

Development requirements

Layer / File(s) Summary
Retain core development tools
requirements-dev.txt
The requirements retain tox, pytest, and flake8. They remove pytest-cov, dataclasses, scanoss>=1.45.0, importlib-metadata, and pytest-xdist.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 1548d

This localized development-dependency cleanup introduces no actionable merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the removal of development requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch req_dev

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.

@soimkim

soimkim commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

삭제 패키지

패키지 이유
scanoss pyproject.toml dependencies에 이미 있음
dataclasses Python 3.10+ 표준 라이브러리 (프로젝트 최소 3.10)
importlib-metadata 3.10+는 importlib.metadata 사용; pip install -e . 시 scanoss 등에서 transitive로 설치됨
pytest-cov tox/CI에서 미사용
pytest-xdist tox/CI에서 미사용

@soimkim
soimkim merged commit a32f535 into main Aug 20, 2026
8 checks passed
@soimkim
soimkim deleted the req_dev branch August 20, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore [PR/Issue] Refactoring, maintenance the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant