civisanalytics / civis-python

Civis API Python Client
BSD 3-Clause "New" or "Revised" License
34 stars 26 forks source link

[CIVIS-6486] ENH `civis.parallel` compatible with joblib >= v1.3.0 #488

Closed jacksonlee-civis closed 5 months ago

jacksonlee-civis commented 5 months ago

Major changes from joblib v1.2.0 to v1.3.0 are incompatible with civis-python. This issue was first reported in #468, and was temporarily fixed by pinning joblib at < v1.3.0 in #469. This pull request updates civis-python so that now it works with joblib >= v1.3.0. After this pull request is merged, I will not make a new civis-python release yet (I have one or two more pull requests in mind before another release). For this pull request, copying notes from the changelog being updated:

Changed

Fixed

Security


jacksonlee-civis commented 5 months ago

Looks like we have a failing test, reverting review for now: Build

Oh, it's one of those flaky tests for retries that are unrelated to this PR. I re-ran the build and it succeeded.