Skip to content

Add [dev] extras to setup.py; remove stale slodels comment - #852

Draft
gabeweisz with Copilot wants to merge 2 commits into
mainfrom
copilot/remove-dev-mode-suggestion
Draft

Add [dev] extras to setup.py; remove stale slodels comment#852
gabeweisz with Copilot wants to merge 2 commits into
mainfrom
copilot/remove-dev-mode-suggestion

Conversation

Copilot AI commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

README.md documents pip install -e .[dev] for local development, but setup.py had no dev extra defined, making that instruction non-functional. A leftover comment inside extras_require also incorrectly implied slodels required a custom index.

Changes

  • setup.py: Remove the outdated # To install slodels, use a custom index comment from the comparative extra
  • setup.py: Add a dev extras group with the dependencies used in CI, pinned to match workflow versions:
"dev": [
    "pytest",
    "black==26.1.0",
],

pip install -e .[dev] now resolves correctly and matches what the lint and regression-test workflows install.

Copilot AI linked an issue Jul 27, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Update README to remove dev mode installation suggestion Add [dev] extras to setup.py; remove stale slodels comment Jul 27, 2026
Copilot AI requested a review from gabeweisz July 27, 2026 20:32
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.

Dev Mode Support

3 participants