Skip to content

chore: remove rich dependency to stabilize installs#2

Merged
nipudarsh merged 5 commits intomainfrom
codex/create-production-ready-angel-project-byvl4r
Jan 29, 2026
Merged

chore: remove rich dependency to stabilize installs#2
nipudarsh merged 5 commits intomainfrom
codex/create-production-ready-angel-project-byvl4r

Conversation

@nipudarsh
Copy link
Owner

Motivation

  • Tests in CI were failing during dependency installation due to the optional rich runtime dependency, so runtime dependencies were reduced to improve install reliability in constrained environments.

Description

  • Removed rich from runtime dependencies in pyproject.toml and requirements.txt to avoid install-time failures.
  • Replaced rich imports and Console usage with click.echo in CLI modules to preserve UX without the extra dependency, touching angel_cli/cli.py, angel_cli/commands/auth.py, angel_cli/commands/tools.py, angel_cli/commands/repos.py, angel_cli/commands/maintenance.py, and angel_cli/commands/config.py.
  • Committed the changes under the message chore: drop rich dependency so the package metadata and CLI are consistent with the simplified dependency set.

Testing

  • No automated tests were executed locally because dependency installation is blocked in the environment, so pytest was not run here (tests remain in the repo and will run in CI once dependencies install successfully).

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1f1eb691cd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@nipudarsh nipudarsh merged commit 4e0440a into main Jan 29, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant