only install needed packages

delvtech · Jan 18, 2024 · 756029b · 756029b
1 parent dd17858
commit 756029b
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 14 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -26,8 +26,9 @@ jobs:
         run: python -m pip install --upgrade pip
 
       - name: install requirements
+        # include toml because a file in scripts/ needs it.
         run: |
-          python -m pip install --upgrade .[all]
+          python -m pip install --upgrade .[test] && python -m pip install toml
 
       - name: run black
         run: python -m black --config pyproject.toml --check --diff .

diff --git a/.github/workflows/static.yml b/.github/workflows/static.yml
@@ -27,7 +27,7 @@ jobs:
 
       - name: install requirements
         run: |
-          python -m pip install --upgrade .[all]
+          python -m pip install --upgrade .[test]
 
       - name: get all Python files
         id: list_files

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -27,7 +27,7 @@ jobs:
 
       - name: install requirements
         run: |
-          python -m pip install --upgrade .[all]
+          python -m pip install --upgrade .[test]
 
       - name: Install Foundry
         uses: foundry-rs/foundry-toolchain@v1

diff --git a/pyproject.toml b/pyproject.toml
@@ -23,17 +23,7 @@ pypechain = "pypechain.main:main"
 
 [project.optional-dependencies]
 # This flag installs all dependencies and should be ran when installing this package
-all = ["pypechain[build,test]"]
-build = [
-    "build",
-    "flit",
-    "request",
-    "setuptools",
-    "toml",
-    "toml",
-    "twine",
-    "wheel",
-]
+all = ["pypechain[test,build]"]
 test = [
     "coverage",
     "factory-boy",
@@ -42,6 +32,7 @@ test = [
     "pytest",
     "pytest-snapshot",
 ]
+build = ["build", "flit", "request", "setuptools", "toml", "twine", "wheel"]
 
 [project.urls]
 "Homepage" = "https://github.com/delvtech/pypechain"