Better organize lint and test dependencies (aio-libs#636)

lysnikolaou · pre-commit-ci[bot] · web-flow · commit c28f32d6816c · 2025-03-17T09:52:05.000-10:00
* Better organize lint and test dependencies - Rename ci deps to test deps to only install test deps in test CI job. - Move build, pre-commit, twine and types-setuptools to lint deps - Move tox to dev dep since it's never used in CI - Move cython into its own file so that it can be used as a constraint - Fix CI jobs and Makefile to use the appropriate requirement files * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Add news blurb * towncrier is not needed for in test deps * Remove version constraints from build and twine * Update CHANGES/636.contrib.rst * Update CHANGES/636.contrib.rst --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
diff --git a/.github/workflows/ci-cd.yml b/.github/workflows/ci-cd.yml
@@ -92,8 +92,6 @@ jobs:
     - name: Install core libraries for build
       run: python -Im pip install build
     - name: Build sdists and pure-python wheel
-      env:
-        PIP_CONSTRAINT: requirements/ci.txt
       run: python -Im build --config-setting=pure-python=true
     - name: Determine actual created filenames
       id: dist-filenames-detection
@@ -240,7 +238,7 @@ jobs:
     - name: Install dependencies
       uses: py-actions/py-dependency-install@v4
       with:
-        path: requirements/ci.txt
+        path: requirements/test.txt
     - name: Determine pre-compiled compatible wheel
       env:
         # NOTE: When `pip` is forced to colorize output piped into `jq`,
diff --git a/.github/workflows/reusable-linters.yml b/.github/workflows/reusable-linters.yml
@@ -59,7 +59,7 @@ jobs:
     - name: Install dependencies
       uses: py-actions/py-dependency-install@v4
       with:
-        path: requirements/ci.txt
+        path: requirements/lint.txt
     - name: Self-install
       run: |
         pip install . --config-settings=pure-python=true
@@ -86,7 +86,6 @@ jobs:
         make doc-spelling
     - name: Prepare twine checker
       run: |
-        pip install -U build twine
         python -m build --config-setting=pure-python=true
     - name: Run twine checker
       run: |
diff --git a/CHANGES/636.contrib.rst b/CHANGES/636.contrib.rst
@@ -0,0 +1,2 @@
+Organized dependencies into test and lint dependencies so that no
+unnecessary ones are installed during CI runs -- by :user:`lysnikolaou`.
diff --git a/Makefile b/Makefile
@@ -6,7 +6,7 @@ SRC = frozenlist tests setup.py
 all: test
 
 .install-cython:
-	pip install -r requirements/ci.txt
+	pip install -r requirements/cython.txt
 	touch .install-cython
 
 frozenlist/%.c: frozenlist/%.pyx
@@ -15,7 +15,7 @@ frozenlist/%.c: frozenlist/%.pyx
 cythonize: .install-cython $(PYXS:.pyx=.c)
 
 .install-deps: $(shell find requirements -type f)
-	pip install -r requirements/ci.txt
+	pip install -r requirements/lint.txt -r requirements/test.txt
 ifndef CI
 	pre-commit install
 endif
diff --git a/requirements/ci.txt b/requirements/ci.txt
diff --git a/requirements/cython.txt b/requirements/cython.txt
@@ -0,0 +1 @@
+cython==3.0.6
diff --git a/requirements/dev.txt b/requirements/dev.txt
@@ -1,2 +1,4 @@
 -e . --config-settings=pure-python=false
--r ci.txt
+-r test.txt
+-r lint.txt
+tox==4.11.4
diff --git a/requirements/lint.txt b/requirements/lint.txt
@@ -0,0 +1,4 @@
+build
+pre-commit==3.5.0
+twine
+types-setuptools==75.8.0.20250210
diff --git a/requirements/test.txt b/requirements/test.txt
@@ -0,0 +1,4 @@
+-r cython.txt
+coverage==7.6.1
+pytest==7.4.3
+pytest-cov==4.1.0

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+Organized dependencies into test and lint dependencies so that no`
	`2`	+unnecessary ones are installed during CI runs -- by :user:`lysnikolaou`.
-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +build
 +pre-commit==3.5.0
 +twine
 +types-setuptools==75.8.0.20250210