Skip to content

Conversation

tiran
Copy link
Contributor

@tiran tiran commented Apr 24, 2024

Changes

Which issue is resolved by this Pull Request:
Resolves #983

Description of your changes:

  • use tox-gh to run tests for each Python env
  • use setup-python's pip cache instead of our own home-grown solution. It's nearly as fast as caching a tox env and less brittle.
  • cache 4GB model from huggingface
  • test on Ubuntu / Python 3.9, 3.10, 3.11, and 3.12
  • test on macOS ARM64 / Python 3.11 (only unit tests, functional shell script does not work on macOS)

@tiran tiran force-pushed the ci-py-matrix branch 9 times, most recently from c904191 to 75efa9e Compare April 25, 2024 10:16
@tiran tiran changed the title Draft: Test with multiple Python versions Test with multiple Python versions Apr 25, 2024
@tiran tiran marked this pull request as ready for review April 25, 2024 10:16
@github-actions github-actions bot added the testing Relates to testing label Apr 25, 2024
Copy link
Member

@russellb russellb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great. It just needs a rebase.

- use `tox-gh` to run tests for each Python env
- use `setup-python`'s pip cache

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@russellb
Copy link
Member

I did the rebase. Now CI just needs to run again to make sure it's all still green.

@russellb
Copy link
Member

all green!

@russellb russellb merged commit 88ba3ec into instructlab:main Apr 26, 2024
russellb added a commit to russellb/instructlab that referenced this pull request Apr 29, 2024
When I merged instructlab#984, I failed to ensure the mergify config was updated
accordingly to reflect the updated set of test jobs.

Signed-off-by: Russell Bryant <rbryant@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: validate unit tests with python3.9
3 participants