Skip to content

Commit

Permalink
Update dependencies to a post-Dagster world
Browse files Browse the repository at this point in the history
Spine Engine doesn't use Dagster anymore, so we need not restrict
spinedb_api dependencies anymore.

Developer and documentation dependencies can now be installed
by a single dev-requirements.txt file.

Re spine-tools/Spine-Toolbox#2522
  • Loading branch information
soininen committed May 15, 2024
1 parent 2e5576f commit 6d0313f
Show file tree
Hide file tree
Showing 7 changed files with 21 additions and 29 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/run_unit_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ jobs:
PYTHONUTF8: 1
run: |
python -m pip install --upgrade pip
python -m pip install .[dev]
python -m pip install .
python -m pip install -r dev-requirements.txt
- name: List packages
run:
python -m pip list
Expand Down Expand Up @@ -89,14 +90,11 @@ jobs:
PYTHONUTF8: 1
run: |
python -m pip install --upgrade pip
python -m pip install -r requirements.txt
python -m pip install -e .
python -m pip install -r dev-requirements.txt
- name: List packages
run:
python -m pip list
- name: Install python3 kernelspecs
run: |
python -m pip install ipykernel
python -m ipykernel install --user
- name: Run tests
run: |
if [ "$RUNNER_OS" != "Windows" ]; then
Expand Down Expand Up @@ -134,7 +132,8 @@ jobs:
PYTHONUTF8: 1
run: |
python -m pip install --upgrade pip
python -m pip install -r requirements.txt
python -m pip install -e .
python -m pip install -r dev-requirements.txt
- name: List packages
run:
python -m pip list
Expand Down
10 changes: 4 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,21 +36,19 @@ To install the latest development version use the Git repository url:
## Building the documentation

Source files for the documentation can be found in `docs/source` directory. In order to
build the HTML docs, you need to install the additional documentation building requirements
build the HTML docs, you need to install the developer dependencies
by running:

$ pip install -r dev-requirements.txt
$ pip install -e .[dev]

This installs Sphinx (among other things), which is required in building the documentation.
When Sphinx is installed, you can build the HTML pages from the source files by running:

> docs\make.bat html
> bin\build_doc.bat

or

$ pushd docs
$ make html
$ popd
$ bin/build_doc.py

depending on your operating system.

Expand Down
7 changes: 7 additions & 0 deletions dev-requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
coverage[toml]
pyperf
sphinx
sphinx-autoapi
sphinx_rtd_theme
recommonmark
black == 21.12b
9 changes: 0 additions & 9 deletions docs/requirements.txt

This file was deleted.

5 changes: 4 additions & 1 deletion docs/source/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,10 @@ def setup(sphinx):
# -- Options for intersphinx extension ---------------------------------------

# Example configuration for intersphinx: refer to the Python standard library.
intersphinx_mapping = {"https://docs.python.org/": None, "https://docs.sqlalchemy.org/en/13/": None}
intersphinx_mapping = {
"python": ("https://docs.python.org/", None),
"sqlalchemy": ("https://docs.sqlalchemy.org/en/13/", None)
}

# -- Options for todo extension ----------------------------------------------

Expand Down
5 changes: 0 additions & 5 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,8 @@ dependencies = [
# v1.4 does not pass tests
"sqlalchemy >=1.3, <1.4",
"alembic >=1.7",
"faker >=8.1.2",
"datapackage >=1.15.2",
"python-dateutil >=2.8.1",
# v1.22 requires Python 3.8 or later
"numpy >=1.20.2",
"scipy >=1.7.1",
"openpyxl >=3.0.7, !=3.1.1",
Expand All @@ -33,9 +31,6 @@ dependencies = [
[project.urls]
Repository = "https://github.com/spine-tools/Spine-Database-API"

[project.optional-dependencies]
dev = ["coverage[toml]", "pyperf"]

[build-system]
requires = ["setuptools>=64", "setuptools_scm[toml]>=6.2", "wheel", "build"]
build-backend = "setuptools.build_meta"
Expand Down
1 change: 0 additions & 1 deletion requirements.txt

This file was deleted.

0 comments on commit 6d0313f

Please sign in to comment.