Merge pull request #82 from janelia-cellmap/feature-updates #288

	name: CI

	# Run on every PR (so feature-branch work is verified) and on direct
	# pushes to main (so post-merge state is verified). Feature-branch
	# pushes WITHOUT an open PR don't trigger -- open a PR to get CI.
	# This avoids the double-firing (push + pull_request) we saw before.
	on:
	push:
	branches: [main]
	pull_request:

	jobs:
	test:
	runs-on: ${{ matrix.os }}
	strategy:
	# Don't kill the linux run on a mac hiccup (or vice versa); we want
	# the per-platform signal to come through independently.
	fail-fast: false
	matrix:
	os: [ubuntu-latest, macos-latest]
	python-version: ['3.12']

	steps:
	- uses: actions/checkout@v3

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

	- name: Upgrade pip and install build tools
	run: \|
	python -m pip install --upgrade pip
	pip install numpy Cython setuptools wheel build

	- name: Install project and dev dependencies
	run: pip install .[dev]

	- name: Run tests with coverage
	run: \|
	pytest --cov=cellmap_analyze --cov-report=xml

	- name: Upload coverage report to Codecov
	# Only upload once per push (from linux) to avoid double-reporting.
	if: matrix.os == 'ubuntu-latest'
	uses: codecov/codecov-action@v4.2.0
	with:
	files: ./coverage.xml
	token: ${{ secrets.CODECOV_TOKEN }}

Provide feedback