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

Stop overwriting preload function #196

Merged
merged 2 commits into from
Nov 2, 2021
Merged

Stop overwriting preload function #196

merged 2 commits into from
Nov 2, 2021

Conversation

jvfe
Copy link
Contributor

@jvfe jvfe commented Nov 2, 2021

I used the first code snippet in the issue to test it, seems to be working fine and as intended in both transcrypt and pyodide modes.

Fixes both pyodide and transcrypt modes so the preload()
function can work as intended.
Copy link
Owner

@berinhard berinhard left a comment

Choose a reason for hiding this comment

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

Thanks a lot for such fast PR @jvfe 🍾 🍾 🍾

The code looks good but the only think I'll ask for is an update in the CHANGELOG.md linking to this PR (examples here).

@berinhard
Copy link
Owner

And once I merge this, I'll release 0.7.1 with this and the other bugfixes

@jvfe
Copy link
Contributor Author

jvfe commented Nov 2, 2021

Thanks a lot for such fast PR @jvfe champagne champagne champagne

The code looks good but the only think I'll ask for is an update in the CHANGELOG.md linking to this PR (examples here).

I always forget to update the changelog, sorry. Done, thanks!

@berinhard berinhard merged commit df93918 into berinhard:develop Nov 2, 2021
@jvfe jvfe deleted the fix/issue_132 branch November 2, 2021 17:09
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.

preload() not working as expected
2 participants