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

Improve python github workflows #779

Merged
merged 3 commits into from
Nov 1, 2024
Merged

Improve python github workflows #779

merged 3 commits into from
Nov 1, 2024

Conversation

reyammer
Copy link
Collaborator

@reyammer reyammer commented Nov 1, 2024

This PR consolidates the python-related workflows and package build scripts:

  • .github/workflows/python-test-suite.yml now tests everything it can that does NOT need the wheels. This includes testing the magika rust binary and the magika python package.
  • maturin.yml is now THE GH workflow in which we build the wheels, test them, and upload them as artifacts.
  • removed all other python-related GH workflows, as the python packages/wheels are now created and tested in maturin.yml.
  • removed the build_python_package.py script; we build the wheels via maturin.yml.

@reyammer reyammer force-pushed the py-improve-gh-workflows branch from ce725c3 to e9642f4 Compare November 1, 2024 15:07
@reyammer reyammer requested a review from ia0 November 1, 2024 15:47
@reyammer reyammer marked this pull request as ready for review November 1, 2024 15:48
@reyammer reyammer requested a review from invernizzi as a code owner November 1, 2024 15:48
@reyammer
Copy link
Collaborator Author

reyammer commented Nov 1, 2024

The resulting linux wheel is still not pip installable on ubuntu 20.04 (see #781), but this PR is a good clean up anyways.

@reyammer reyammer merged commit 348a34d into main Nov 1, 2024
24 checks passed
@reyammer reyammer deleted the py-improve-gh-workflows branch November 1, 2024 16:13
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.

1 participant