Skip to content

Simplify js2python string conversion code a bit#1679

Merged
hoodmane merged 1 commit intopyodide:mainfrom
hoodmane:simpler-js2py-string-conversion
Jul 1, 2021
Merged

Simplify js2python string conversion code a bit#1679
hoodmane merged 1 commit intopyodide:mainfrom
hoodmane:simpler-js2py-string-conversion

Conversation

@hoodmane
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

Assuming CI is OK, thanks!

@hoodmane
Copy link
Copy Markdown
Member Author

There are problems in Firefox in the CI. I do not yet understand why.

@hoodmane
Copy link
Copy Markdown
Member Author

hoodmane commented Jul 1, 2021

Problem was just a sporadic failure.

@hoodmane hoodmane merged commit 9ec832b into pyodide:main Jul 1, 2021
@hoodmane hoodmane deleted the simpler-js2py-string-conversion branch July 1, 2021 02:42
@hoodmane
Copy link
Copy Markdown
Member Author

hoodmane commented Jul 1, 2021

We might want to revert this, it seems to be inconsistently breaking up firefox-core tests on CI (cannot reproduce test failures locally though).

@rth
Copy link
Copy Markdown
Member

rth commented Jul 3, 2021

Yes, it does look like there are more timeouts after the last few commits. Thoughg it could be also due to CircleCI having larger loads and being slower as well.

hamlet4401 pushed a commit to tytgatlieven/pyodide that referenced this pull request Jul 3, 2021
hoodmane pushed a commit to hoodmane/pyodide that referenced this pull request Jul 3, 2021
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