Declare the dev extra the Readme installs; test pull requests in CI

.github/workflows/ci-build.yml

@@ -1,6 +1,7 @@
 name: Build unstable
 
-on: [push]
+# pull_request so that contributions from forks are tested too
+on: [push, pull_request]
 
 concurrency: 
   group: unstable
@@ -30,8 +31,8 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install --upgrade flake8 pytest pycodestyle pytest-cov pytest-mock "mypy==2.3.*"
-          if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+          pip install --upgrade pycodestyle "mypy==2.3.*"
+          pip install -e ".[dev]"
       - name: Lint with flake8
         run: |
           # stop the build if there are Python syntax errors or undefined names

.gitignore

@@ -21,3 +21,6 @@ doc/
 build/
 dist/
 grobid_client_python.egg-info/
+.coverage
+coverage.xml
+.pytest_cache/

Readme.md

@@ -796,9 +796,6 @@ source venv/bin/activate  # On Windows: venv\Scripts\activate
 
 # Install in development mode with test dependencies
 pip install -e .[dev]
-
-# Install pre-commit hooks (optional)
-pre-commit install
 ```
 
 ### Creating a New Release

pyproject.toml

@@ -20,13 +20,26 @@ maintainers = [
 ]
 description = "Simple python client for GROBID REST services"
 readme = "Readme.md"
+requires-python = ">=3.8"
 
 dynamic = ['version', "dependencies"]
 
 [project.optional-dependencies]
 # Streaming inputs directly from S3 (s3:// URIs/prefixes). smart_open provides
 # a seekable, range-streamed reader so remote zips are never fully downloaded.
 s3 = ["smart_open[s3]>=6.0", "boto3"]
+# Everything the test-suite needs (moto mocks S3 for tests/test_s3.py, which
+# is skipped without it). `pip install -e .[dev]` as the Readme says.
+dev = [
+    "pytest",
+    "pytest-cov",
+    "pytest-mock",
+    "mypy",
+    "flake8",
+    "moto[s3]>=4.0",
+    "smart_open[s3]>=6.0",
+    "boto3",
+]
 
 [tool.setuptools.dynamic]
 dependencies = {file = ["requirements.txt"]}