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

Build updates #861

Merged
merged 11 commits into from
Apr 25, 2024
Merged

Build updates #861

merged 11 commits into from
Apr 25, 2024

Conversation

kbattocchi
Copy link
Collaborator

This PR makes several improvements to our build process and package metadata:

  • Switches to a new library uv for installing dependencies on our test machines, which saves ~1-2 minutes on each machine
  • Adds Python 3.12 to our test matrix
  • Enables compatibility with dowhy 3.12
  • Makes it so that the [all] and [tf] extras only apply restrictions to the versions of dependencies that we use when necessary (i.e. when we're using Python 3.8), which would otherwise cause incompatibility with the versions supported by Python 3.12
  • Switches away from using setup.py pytest to run tests, which has been deprecated for a long time, in favor of calling pytest directly
  • Updates the LKG files to include more recent versions of dependencies
  • Migrates to updated versions of the GitHub Actions artifact API, which eliminates the warnings we were getting
  • Migrates more of our package's configuration from setup.cfg to pyproject.toml

@kbattocchi kbattocchi requested a review from fverac March 11, 2024 16:14
@kbattocchi kbattocchi force-pushed the kebatt/updateCI branch 2 times, most recently from f5f5e9d to 340e3a5 Compare March 11, 2024 16:43
Copy link
Collaborator

@fverac fverac left a comment

Choose a reason for hiding this comment

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

looks good!

Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
@kbattocchi kbattocchi force-pushed the kebatt/updateCI branch 2 times, most recently from 7fb83f3 to 3f5d8a1 Compare April 18, 2024 18:57
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
@kbattocchi kbattocchi force-pushed the kebatt/updateCI branch 4 times, most recently from e0be43a to 4dde5b0 Compare April 24, 2024 18:23
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
@kbattocchi kbattocchi merged commit a0054d1 into main Apr 25, 2024
97 checks passed
@kbattocchi kbattocchi deleted the kebatt/updateCI branch April 25, 2024 06:19
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