Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make agent dependencies optional #95

Closed
wants to merge 10 commits into from

Conversation

xingyaoww
Copy link

@xingyaoww xingyaoww commented Nov 21, 2024

This PR adds a core installation option that excludes agent-related dependencies. Changes include:

  1. Keep aider-chat in main dependencies (so pip install commit0 installs everything)
  2. Add core optional dependencies group that excludes aider-chat
  3. Move agent package to commit0/optional/agent
  4. Add build hook to include/exclude agent package based on installation type

This allows users to:

  • Install everything (default): pip install commit0
    • Includes agent package
    • Includes aider-chat dependency
    • Provides commit0 command
  • Install without agent support: pip install commit0[core]
    • Excludes agent package
    • Excludes aider-chat dependency
    • Provides same commit0 command

This is a simpler approach than previous PRs as it:

  • Keeps all code in the same package
  • Uses Python's built-in optional dependencies
  • Makes the full installation the default
  • Uses the same command (commit0) for both installations

@wenting-zhao
Copy link
Collaborator

merged in #96

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.

3 participants