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

CI: Pin Cython on CI #54757

Merged
merged 1 commit into from
Aug 25, 2023
Merged

CI: Pin Cython on CI #54757

merged 1 commit into from
Aug 25, 2023

Conversation

phofl
Copy link
Member

@phofl phofl commented Aug 25, 2023

@phofl phofl requested a review from mroeschke as a code owner August 25, 2023 14:48
@phofl phofl added this to the 2.1 milestone Aug 25, 2023
@phofl phofl added the CI Continuous Integration label Aug 25, 2023
@WillAyd WillAyd merged commit ee51d7b into pandas-dev:main Aug 25, 2023
37 checks passed
@WillAyd
Copy link
Member

WillAyd commented Aug 25, 2023

Thanks @phofl

@lithomas1
Copy link
Member

lithomas1 commented Aug 25, 2023

Looks like python dev needs to be pinned as well, PR incoming.

@@ -25,7 +25,7 @@ dependencies:
- pip

- pip:
- "cython"
- "cython<=3.0.1"
Copy link
Member

Choose a reason for hiding this comment

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

Don't we not want 3.0.1?

@mroeschke
Copy link
Member

I think we also need to pin cython in .github/workflows/unit-tests.yml

lithomas1 pushed a commit that referenced this pull request Aug 26, 2023
Backport PR #54757: CI: Pin Cython on CI

Co-authored-by: Patrick Hoefler <61934744+phofl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants