Remove print statement and add linting (#45) #126

	# This workflow will install Python dependencies, run tests and lint with a variety of Python versions
	# For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions

	name: test

	on:
	push:
	branches: [main]
	pull_request:
	branches: [main]

	jobs:
	lint_and_test:
	runs-on: ${{ matrix.os }}
	strategy:
	matrix:
	python-version: ["3.8", "3.10"] # 3.10 is converted to 3.1 if not a string
	os: [ubuntu-latest, macos-latest, windows-latest]
	env:
	OS: ${{ matrix.os }}
	PYTHON_VERSION: ${{ matrix.python-version }}
	steps:
	- uses: actions/checkout@v2
	- name: Set up Python ${{ matrix.python-version }}
	uses: actions/setup-python@v1
	with:
	python-version: ${{ matrix.python-version }}
	- name: Install dependencies
	run: \|
	python -m pip install --upgrade pip
	pip install .[test]
	- name: Check code formatting and linting
	if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.8' # linting should occur only once in the loop
	run: \|
	make lint
	- name: Test with pytest and generate coverage report
	run: \|
	pytest --cov=./ --cov-report=xml
	- name: Upload coverage report to Codecov
	uses: codecov/codecov-action@v3
	if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.8' # upload should occur only once in the loop
	with:
	token: ${{ secrets.CODECOV_TOKEN }} # token is not mandatory but make access more stable
	file: ./coverage.xml
	env_vars: OS,PYTHON
	fail_ci_if_error: true

Provide feedback