Skip to content

Consolidate dev dependencies in pyproject.toml and eliminate redundancy#55

Draft
Copilot wants to merge 2 commits intofeat/quick-setupfrom
copilot/sub-pr-53-again
Draft

Consolidate dev dependencies in pyproject.toml and eliminate redundancy#55
Copilot wants to merge 2 commits intofeat/quick-setupfrom
copilot/sub-pr-53-again

Conversation

Copy link
Contributor

Copilot AI commented Dec 18, 2025

Addresses feedback on PR #53 regarding duplicate dependency declarations between requirements-dev.txt and pyproject.toml[project.optional-dependencies.dev], which caused redundant installations and potential version conflicts.

Changes

  • Added docs extra in pyproject.toml: Separated Sphinx documentation dependencies from dev tools
  • Refactored requirements-dev.txt: Now references pyproject.toml extras via -e .[dev,docs] instead of duplicating dependency declarations
  • Updated setup scripts: Both setup_dev.sh and setup_dev.bat now install .[dev,docs] directly, eliminating the two-step installation process

Before

# setup_dev.sh installed dependencies twice
pip install -e ".[dev]" --quiet
pip install -r requirements-dev.txt --quiet  # Reinstalled pytest, black, mypy, flake8, isort + sphinx

After

# Single installation with clear separation
pip install -e ".[dev,docs]" --quiet

All dependencies now have a single source of truth in pyproject.toml. The requirements-dev.txt file remains for backwards compatibility but references the pyproject.toml extras.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

…ts-dev.txt

Co-authored-by: omsherikar <180152315+omsherikar@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on quick setup improvements Consolidate dev dependencies in pyproject.toml and eliminate redundancy Dec 18, 2025
Copilot AI requested a review from omsherikar December 18, 2025 05:58
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.

2 participants