Merge pull request #138 from mdevolde/uv

feat (build_and_publish): moved build and publish script in Makefiles, used uv instead of twine

Author: mdevolde

.github/workflows/test.yml

@@ -38,7 +38,7 @@ jobs:
 
     - name: Install dependencies & build package
       run: |
-        uv pip install setuptools wheel build pytest pytest-cov pytest-xdist --system
+        uv pip install setuptools wheel build pytest pytest-cov pytest-rerunfailures pytest-xdist --system
         uv build
         uv pip install dist/*.whl --system
 

MANIFEST.in

@@ -20,8 +20,8 @@ prune tests/
 
 exclude .gitignore
 exclude .readthedocs.yaml
-exclude build_and_publish.sh
 exclude CONTRIBUTING.md
+exclude coverage-badge.svg
 exclude make.bat
 exclude Makefile
 exclude pytest.ini

Makefile

@@ -1,7 +1,7 @@
-.PHONY: default check test doc
+.PHONY: default check test doc publish
 
 default:
-	@echo "Usage: make [check|test|doc]"
+	@echo "Usage: make [check|test|doc|publish]"
 	@exit 1
 
 check:
@@ -14,4 +14,10 @@ test:
 	uvx --with defusedxml genbadge coverage --input-file coverage.xml --silent
 doc:
 	source ./.venv/bin/activate && uv run sphinx-apidoc -o docs/source/references language_tool_python
-	source ./.venv/bin/activate && cd ./docs && make html
+	source ./.venv/bin/activate && cd ./docs && make html
+
+publish:
+	rm -rf dist/ language_tool_python.egg-info/
+	uv build
+	uvx twine check dist/*
+	uv publish

build_and_publish.sh

@@ -3,8 +3,9 @@
 if "%1"=="check" goto check
 if "%1"=="test" goto test
 if "%1"=="doc" goto doc
+if "%1"=="publish" goto publish
 
-echo Usage: make.bat [check^|test^|doc]
+echo Usage: make.bat [check^|test^|doc^|publish]
 exit /b 1
 
 :check
@@ -19,10 +20,29 @@ exit /b %errorlevel%
 
 :test
 pytest
+if errorlevel 1 exit /b %errorlevel%
+
 uvx --with defusedxml genbadge coverage --input-file coverage.xml --silent
 exit /b %errorlevel%
 
 :doc
 uv sync --group tests --group docs --group types
+
 call .venv\Scripts\activate && uv run sphinx-apidoc -o docs\source\references language_tool_python
+if errorlevel 1 exit /b %errorlevel%
+
 call .venv\Scripts\activate && call docs\make.bat html
+exit /b %errorlevel%
+
+:publish
+if exist dist\ rmdir /s /q dist\
+if exist language_tool_python.egg-info\ rmdir /s /q language_tool_python.egg-info\
+
+uv build
+if errorlevel 1 exit /b %errorlevel%
+
+uvx twine check .\dist\*
+if errorlevel 1 exit /b %errorlevel%
+
+uv publish
+exit /b %errorlevel%

make.bat

@@ -43,6 +43,7 @@ tests = [
     "pytest",
     "pytest-xdist",
     "pytest-cov",
+    "pytest-rerunfailures",
     "pytest-runner"
 ]
 

pyproject.toml

@@ -1,7 +1,16 @@
 """Tests for the public API functionality."""
 
+import os
+
+import pytest
+
 from language_tool_python.exceptions import RateLimitError
 
+pytestmark = pytest.mark.skipif(
+    os.getenv("GITHUB_ACTIONS") == "true",
+    reason="Skip public API test on CI (depends on external service).",
+)
+
 
 def test_remote_es() -> None:
     """

tests/test_api_public.py

@@ -2,7 +2,10 @@
 
 from typing import Any, Dict, List
 
+import pytest
 
+
+@pytest.mark.flaky(reruns=2)  # type: ignore[misc] # Sometimes LT throws NoClassDefFoundError (500)
 def test_langtool_load() -> None:
     """
     Test the basic functionality of LanguageTool and Match object attributes.