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

This is a test #90

Closed
wants to merge 12 commits into from
Closed

This is a test #90

wants to merge 12 commits into from

Conversation

Ce11an
Copy link
Contributor

@Ce11an Ce11an commented Mar 14, 2024

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.

What is the motivation?

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix

What does this change do?

What is your testing strategy?

Is this related to any issues?

Have you read the Contributing Guidelines?

Ce11an added 12 commits March 10, 2024 16:39
These scripts are replaced by `maturin`
`maturin` simplifies testing
Incorrectly added in previous commit. Also removing extra build jobs. We can see if some bits can be used for the "cross-build" jobs
I have a sneaky suspicions that the jobs failed due to an incorrect path
I cannot get it to work... I do not want to block this PR. This was going to be an addition to the codebase. This PR does not change the current behaviour of the tests.
These were not included previously
@Ce11an Ce11an closed this Mar 14, 2024
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