Skip to content

Conversation

codegen-sh[bot]
Copy link
Contributor

@codegen-sh codegen-sh bot commented Sep 23, 2025

This PR demonstrates what it would look like to remove deptry from the codebase.

What is deptry?

deptry is a Python dependency checker that scans your codebase to find:

  • DEP001: Missing dependencies (imports used but not in dependencies)
  • DEP002: Unused dependencies (dependencies listed but not imported)
  • DEP003: Transitive dependencies (imports that rely on sub-dependencies)
  • DEP004: Misplaced dev dependencies (dev deps used in production code)
  • DEP005: Standard library dependencies (stdlib modules listed as deps)

Current Status

Currently, deptry runs successfully with no issues found:

Scanning 151 files...
Success! No dependency issues found.

Changes Made

This PR removes deptry completely:

  • ✅ Removed from pre-commit hooks (.pre-commit-config.yaml)
  • ✅ Removed from dev dependencies (pyproject.toml)
  • ✅ Removed configuration section ([tool.deptry])
  • ✅ Removed from setup script (scripts/setup.sh)
  • ✅ Updated lockfile (uv.lock)

Should We Remove It?

Arguments for keeping deptry:

  • Catches dependency drift over time
  • Prevents unused dependencies from accumulating
  • Ensures all imports have corresponding dependencies
  • Lightweight and fast (runs in ~4 seconds)

Arguments for removing deptry:

  • Currently reports no issues
  • Adds complexity to pre-commit setup
  • May have false positives that need configuration
  • Other tools (like IDEs) can catch missing imports

Recommendation

Since deptry is currently working well and finding no issues, it suggests your dependency management is already clean. However, it provides ongoing value by preventing future dependency issues. The overhead is minimal, so I'd recommend keeping it unless you have specific issues with it.


This is a demonstration PR - not intended for merge


💻 View my work • 👤 Initiated by Jay Hack • 💬 Initiated in #demoAbout Codegen
⛔ Remove Codegen from PR🚫 Ban action checks

- Remove deptry from pre-commit hooks
- Remove deptry from dev dependencies in pyproject.toml
- Remove deptry configuration section
- Remove deptry from setup script
- Update lockfile to reflect removal
Copy link
Contributor Author

codegen-sh bot commented Sep 23, 2025

🔍 Broken test auto-fixer • Learn more

Check Suite Agent Status Commit Time
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:53 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:55 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:55 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:56 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:57 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 18:55:57 UTC
GitHub Actions Agent Fix ✅ 02d09e2 Sep 23, 19:00:29 UTC

💻 View my work🛑 Stop🚫 Ban all checks

⚙️ Check suite settings

Copy link

codecov bot commented Sep 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

- Remove empty lines in dev-dependencies list
- Remove empty line in setup script
- Remove orphaned deptry configuration lines
- Clean up extra blank lines in config files
Copy link
Contributor Author

codegen-sh bot commented Sep 23, 2025

Fixed failing pre-commit checks on PR #1276

The issue was formatting problems left behind from the deptry removal:

Problems fixed:

  • 🔧 Removed empty line in dev-dependencies list in pyproject.toml
  • 🔧 Removed empty line in scripts/setup.sh
  • 🔧 Removed orphaned deptry configuration lines in pyproject.toml
  • 🔧 Cleaned up extra blank lines in .pre-commit-config.yaml

Commit: 63185d9 - Fix formatting issues from deptry removal

The pre-commit checks should now pass! 🎉

💻 View my work🛑 Stop🚫 Ban all checks • React 👍 or 👎

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.

0 participants