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

Added automatic testing and publishing through Github Actions #4

Merged
merged 20 commits into from
Aug 9, 2023

Conversation

jaspersiebring
Copy link
Owner

@jaspersiebring jaspersiebring commented Aug 8, 2023

Currently includes three Github Actions workflows: (1) quality control check for pull requests (pytest, mypy and pylint), (2) publish to PYPI on merged pull requests and (3) a manual build for development purposes. Also features caching since wxPython takes forever to build (>45m) and doesn't have readily-available wheels for Linux distros on PYPI (see https://wxpython.org/pages/downloads/).

The way we currently cache ensures that small versions bumps or new independent packages don't require entirely new poetry install calls (which dramatically speeds up workflow runs from >45 minutes to ~1 minute). This does have some caveats that we just have to watch out for (e.g. cache bloat/stale dependencies, see actions/setup-python#627).

Edit: Also, all workflows currently run on Github's default runners (ubuntu-latest) so currently only Linux artifacts.
Edit 2: Did an interactive rebase to cut down on some of the redundant commits (e.g. build triggers and tests).

@jaspersiebring jaspersiebring self-assigned this Aug 8, 2023
@jaspersiebring jaspersiebring changed the title Linter fixes (GitHub Actions tests) Added automatic testing and publishing through Github Actions Aug 9, 2023
@jaspersiebring jaspersiebring added the enhancement New feature or request label Aug 9, 2023
@jaspersiebring jaspersiebring merged commit 9628550 into main Aug 9, 2023
@jaspersiebring jaspersiebring deleted the linter branch August 9, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant