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

Adds Pyodide version check to prevent incorrect bundle release #3602

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dom96
Copy link
Collaborator

@dom96 dom96 commented Feb 25, 2025

Just a simple check to ensure an incorrectly build version of the pyodide bundle cannot be released.

  • Rebuild 0.26.0 bundle to ensure this works.

@dom96 dom96 force-pushed the dominik/pyodide-ver-check branch 2 times, most recently from 31efac8 to 00c1e24 Compare February 26, 2025 14:21
@dom96 dom96 requested a review from hoodmane February 26, 2025 14:22
@dom96 dom96 marked this pull request as ready for review February 26, 2025 14:22
@dom96 dom96 requested review from a team as code owners February 26, 2025 14:22
@dom96 dom96 requested a review from npaun February 26, 2025 14:22
Copy link
Contributor

@hoodmane hoodmane left a comment

Choose a reason for hiding this comment

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

Thanks!

@dom96 dom96 force-pushed the dominik/pyodide-ver-check branch 2 times, most recently from 2ed7c19 to 7d6f4af Compare February 27, 2025 10:28
@dom96 dom96 force-pushed the dominik/pyodide-ver-check branch from 7d6f4af to 8f0a15c Compare February 27, 2025 11:34
@dom96
Copy link
Collaborator Author

dom96 commented Feb 27, 2025

Not going to push the rebuilt bundle because it changes jaeger tracing and I'd rather avoid having to create a EW PR to fix that. The important part is that the rebuilt bundle passes the other tests.

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