Add some doc comments, rename an argument #4051

	name: Run tests

	on:
	push:
	branches: [main]

	pull_request:
	branches: [main]

	jobs:
	pre-commit:
	runs-on: ubuntu-latest

	steps:
	- uses: actions/checkout@v4

	- name: Set up Node.js
	uses: actions/setup-node@v4
	with:
	node-version: 19.x
	cache: "npm"

	- name: Install Node dependencies
	run: \|
	npm install

	- name: Run pre-commit
	uses: pre-commit/[email protected]
	with:
	extra_args: --all-files --verbose --show-diff-on-failure

	build:
	runs-on: ${{ matrix.os }}

	strategy:
	matrix:
	os: [windows-latest, ubuntu-latest]
	python-version: ["3.10", "3.12"]
	node-version: [19.x]
	exclude:
	- os: windows-latest
	python-version: "3.10"

	steps:
	- uses: actions/checkout@v4

	- name: Set up Python ${{ matrix.python-version }}
	uses: actions/setup-python@v5
	with:
	python-version: ${{ matrix.python-version }}

	- name: Set up Node.js ${{ matrix.node-version }}
	uses: actions/setup-node@v4
	with:
	node-version: ${{ matrix.node-version }}
	cache: "npm"

	- name: Install Python dependencies
	run: \|
	python -m pip install --upgrade pip
	python -m pip install . --no-deps
	python -m pip install -r requirements.txt
	python -m pip install -r requirements-dev.txt

	# # don't need to install Node dependencies because they are installed by the build hook
	# - name: Install Node dependencies
	# run: \|
	# npm install

	- name: Run Python tests with pytest
	run: \|
	pytest -vv

	- name: Run mypy
	run: \|
	mypy src/fontra/ test-py/

	- name: Run Node tests
	run: \|
	npm test

Provide feedback