Skip to content
This repository was archived by the owner on May 3, 2023. It is now read-only.

CI: Sync Minimum Versions #141

Merged
merged 4 commits into from
Jul 7, 2021
Merged

Conversation

lithomas1
Copy link
Contributor

Copy link
Collaborator

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @lithomas1

@@ -22,13 +22,15 @@ jobs:
py_3.7_32:
PYTHON_VERSION: "3.7"
PYTHON_ARCH: "x86"
NP_BUILD_DEP: "1.16.5"
NIGHTLY_BUILD: "true"
NP_BUILD_DEP: "1.17.3"
Copy link
Collaborator

Choose a reason for hiding this comment

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

we still need to build 1.3.1, 1.3.2, .... etc

we won't be increasing min versions in patch releases, so this needs to reverted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

1.17.3 is the minimum version for 1.3.x though, but it wasn't bumped when the PR that released 1.3 was merged. This PR is just syncing the min versions with what is present on the main repo. See https://github.com/pandas-dev/pandas/blob/1.3.x/pyproject.toml for example which is supported by the docs link in the OP.

Copy link
Collaborator

Choose a reason for hiding this comment

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

1.17.3 is the minimum version for 1.3.x though

thanks. is this PR ready to merge. The failures are unrelated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep. Thanks for the quick reviews :).

NIGHTLY_BUILD: "true"
NP_BUILD_DEP: "1.17.3"
# pandas 1.4 requires py3.8+, remove this job on 1.4 release
NIGHTLY_BUILD: "false"
Copy link
Collaborator

Choose a reason for hiding this comment

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

removing from nightly build though is fine.

- NP_BUILD_DEP="numpy==1.16.5"
- NP_TEST_DEP="numpy==1.16.5"
- NP_BUILD_DEP="numpy==1.17.3"
- NP_TEST_DEP="numpy==1.17.3"
Copy link
Collaborator

Choose a reason for hiding this comment

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

a couple of lines below, we have NP_BUILD_DEP="numpy==1.19.1" for 3.8. Our min on master in pandas (i.e. 1,4) is 1.18.5. Any idea why there is a discrepancy there?

Copy link
Collaborator

Choose a reason for hiding this comment

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

nvm. it's probably for the aarch support.

@simonjayhawkins simonjayhawkins merged commit faa31fc into MacPython:master Jul 7, 2021
@simonjayhawkins
Copy link
Collaborator

Thanks @lithomas1

@lithomas1 lithomas1 deleted the patch-2 branch July 7, 2021 16:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants