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

.github/workflows/ci-sage.yml: New #121

Merged
merged 4 commits into from
Feb 3, 2024
Merged

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Feb 2, 2024

This adds portability and integration testing using the reusable GH Actions workflows of SageMath.

python-flint is added to Sage in sagemath/sage#37224

Based on the discussion in flintlib/flint#1760, it is configured to only run as a weekly cron job and on tags, not on individual PRs.

Preview: https://github.com/mkoeppe/python-flint/actions/runs/7762469133

@oscarbenjamin
Copy link
Collaborator

Are you checking that this works by looking somewhere else rather than the actions in this repo?

It doesn't look like it will do anything here until after it is merged.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 2, 2024

Yes, I invoked it manually in https://github.com/mkoeppe/python-flint/actions/runs/7762469133

@mkoeppe mkoeppe marked this pull request as ready for review February 3, 2024 01:40
@oscarbenjamin
Copy link
Collaborator

Okay, lets merge it and see. Thanks Matthias.

@oscarbenjamin oscarbenjamin merged commit 0aced1f into flintlib:master Feb 3, 2024
22 checks passed
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 3, 2024

Thanks!

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