Skip to content

MAINT: Update Pyodide to 0.25.1 #26185

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

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Conversation

charris
Copy link
Member

@charris charris commented Mar 31, 2024

Closes #26164.

@charris charris added 03 - Maintenance 09 - Backport-Candidate PRs tagged should be backported labels Mar 31, 2024
@charris
Copy link
Member Author

charris commented Mar 31, 2024

Patching pyodide.py is disabled here because it was failing. Not sure what has changed.

@agriyakhetarpal
Copy link
Contributor

Hi, @charris! I was fixing this locally actually, now I see that you've done that already :) Thank you for the fix, maybe I should have commented there earlier, but didn't want to pester people on a weekend. It should now be okay to remove the patch step and the file because the commit that the patch was made out of has been included in the v0.25.1 release. Here it is: pyodide/pyodide@3b540c8

@charris charris force-pushed the update-pyodide-version branch from 81458b4 to 42bbfce Compare March 31, 2024 18:30
@charris
Copy link
Member Author

charris commented Mar 31, 2024

@agriyakhetarpal Thanks for the information. I've removed the patch file and the commented out lines.

Copy link
Contributor

@agriyakhetarpal agriyakhetarpal left a comment

Choose a reason for hiding this comment

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

The build and tests are passing here, so LGTM. Thank you for the quick fix!

@ngoldbaum ngoldbaum merged commit a52c49a into numpy:main Mar 31, 2024
@charris charris deleted the update-pyodide-version branch March 31, 2024 19:29
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: emscripten pyiodide build failing
3 participants