Skip to content

Commit

Permalink
Use optional-dependencies for all dependency groups
Browse files Browse the repository at this point in the history
This moves us away from requirements files and centralizes all
dependency specifications under 'project.optional-dependencies'.
  • Loading branch information
jparise committed Jan 6, 2024
1 parent f0786e5 commit ade5489
Show file tree
Hide file tree
Showing 5 changed files with 23 additions and 14 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,10 @@ jobs:
with:
python-version: ${{ matrix.python-version }}
cache: 'pip'
cache-dependency-path: 'tests/requirements.txt'
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools
python -m pip install -r tests/requirements.txt
pip install -e .
pip install -e .[lint,test,typing]
- name: Lint
run: |
flake8
Expand Down
3 changes: 2 additions & 1 deletion .readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ python:
install:
- method: pip
path: .
- requirements: docs/requirements.txt
extra_requirements:
- docs
2 changes: 0 additions & 2 deletions docs/requirements.txt

This file was deleted.

21 changes: 20 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,29 @@ classifiers = [
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
]
requires-python = ">= 3.8"
dependencies = [
"httpx>=0.23.1",
]
requires-python = ">= 3.8"

[project.optional-dependencies]
docs = [
"Sphinx==7.2.2",
"furo==2023.9.10",
]
lint = [
"black==23.12.0",
"flake8==7.0.0",
"isort==5.13.0",
]
test = [
"pytest==7.4.0",
"pytest-cov==4.1.0",
"respx==0.20.1",
]
typing = [
"mypy==1.8.0",
]

[project.urls]
Homepage = "https://github.com/jparise/vesta"
Expand Down
7 changes: 0 additions & 7 deletions tests/requirements.txt

This file was deleted.

0 comments on commit ade5489

Please sign in to comment.