Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configure tests on Github Actions #18

Merged
merged 4 commits into from
Jan 9, 2024
Merged

Configure tests on Github Actions #18

merged 4 commits into from
Jan 9, 2024

Conversation

sandre35
Copy link
Contributor

@sandre35 sandre35 commented Jan 9, 2024

No description provided.

@@ -0,0 +1,25 @@
name: Launch tests and quality gate

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: Launch tests and quality gate
name: tests and quality targets


jobs:
build:
name: Launch tests on test project

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: Launch tests on test project
name: tests

python-version: ${{ matrix.version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
python -m pip install --upgrade pip setuptools wheel
pip install --upgrade pip setuptools wheel

pytz
readme_renderer
setuptools

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
setuptools
setuptools # needed for docs build (conf.py)

Launching tests using `python setup.py tests`is deprecated. This is why we
introduce pytest.
Build-system info are useless in setup.cfg
@sandre35 sandre35 force-pushed the sandre/project_tools branch from 05f4cb7 to 617ca2d Compare January 9, 2024 16:34
@sandre35 sandre35 merged commit 617ca2d into master Jan 9, 2024
5 checks passed
@sandre35 sandre35 deleted the sandre/project_tools branch January 9, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants