Skip to content

fix(ci): add missing dependencies to requirements-ci.txt#253

Merged
pradeeban merged 1 commit intoControlCore-Project:devfrom
GaneshPatil7517:fix/ci-dependencies
Feb 11, 2026
Merged

fix(ci): add missing dependencies to requirements-ci.txt#253
pradeeban merged 1 commit intoControlCore-Project:devfrom
GaneshPatil7517:fix/ci-dependencies

Conversation

@GaneshPatil7517
Copy link
Copy Markdown
Contributor

@GaneshPatil7517 GaneshPatil7517 commented Feb 11, 2026

Hey pradeeban Sir,

This PR fixes the CI test import errors by adding some missing dependencies.

The CI lint-and-test workflow currently fails with:

ModuleNotFoundError: No module named 'click'

Because of this error the tests are not able to run and the CI checks fail.

To fix this, I added the required dependencies to requirements-ci.txt:

  • click>=8.0.0 – used in tests/test_cli.py
  • rich>=10.0.0 – required for CLI output
  • psutil>=5.8.0 – used by runtime utilities
  • beautifulsoup4 – used in tests/test_graph.py
  • lxml – required as the XML parser

Note: This PR is kept separate from PR #252 (Java literalEval fix) as suggested, so that each PR focuses on a single change.

Add click, rich, psutil, beautifulsoup4, and lxml which are required
by tests/test_cli.py and tests/test_graph.py for test imports to work.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes CI import failures by extending the CI-only Python dependency set so the lint-and-test workflow can import and execute the CLI/test modules successfully.

Changes:

  • Add click, rich, psutil, beautifulsoup4, and lxml to requirements-ci.txt to resolve ModuleNotFoundError during CI runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pradeeban pradeeban merged commit 2666d9e into ControlCore-Project:dev Feb 11, 2026
6 of 12 checks passed
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