Skip to content

Conversation

calad0i
Copy link
Contributor

@calad0i calad0i commented Jul 16, 2024

Description

Cleanup the test yaml generator a bit:

  • All tests (include the optimization api ones) are batched together
  • Prefer to group tests need example-models together

Type of change

  • Housekeeping

Tests

Checklist

  • I have read the guidelines for contributing.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have installed and run pre-commit on the files I edited or added.
  • I have added tests that prove my fix is effective or that my feature works.

@calad0i calad0i requested a review from jmitrevs July 16, 2024 04:07
@calad0i calad0i added the please test Trigger testing by creating local PR branch label Jul 16, 2024
@jmitrevs
Copy link
Contributor

There is no reason why the initial part uses 3.8. It can be swapped for 3.10 or later easily. That part just needs a python environment.

@calad0i calad0i added please test Trigger testing by creating local PR branch and removed please test Trigger testing by creating local PR branch labels Jul 16, 2024
@jmitrevs jmitrevs merged commit 5c0c4e6 into fastmachinelearning:main Jul 16, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please test Trigger testing by creating local PR branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants