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

Set browserslist as same as the Streamlit frontend package so that Babel can transpile some modern syntax that will be introduced to the Streamlit codebase #470

Closed
wants to merge 1 commit into from

Conversation

whitphx
Copy link
Owner

@whitphx whitphx commented Feb 5, 2023

This is a part of upgrading the streamlit submodule from v1.13.0 to v1.17.0.
A modern syntax has been introduced to streamlit/frontend since v1.17.0 such as this and it causes an error at Babel transpiling with the current browser compatibility settings on @stlite/mountable and @stlite/desktop.
So we upgraded the settings to be same as the one defined at streamlit/frontend/package.json so that it can be transpiled.

@whitphx whitphx changed the title Set browserslist as same as the Streamlit frontend package so that Babel can transpile some modern syntax that will be introduced to the Streamlit codebase such as https://github.com/streamlit/streamlit/pull/5913/files#diff-845917f3a07167e741db444532fae1e083d5b9f84ac8e8e38d3a34818a311ad8R242 Set browserslist as same as the Streamlit frontend package so that Babel can transpile some modern syntax that will be introduced to the Streamlit codebase Feb 5, 2023
@github-actions github-actions bot temporarily deployed to Preview: (fix/browserslist) February 5, 2023 11:37 Inactive
@whitphx whitphx marked this pull request as draft February 6, 2023 06:11
@whitphx
Copy link
Owner Author

whitphx commented Feb 6, 2023

This PR is incomplete at this point because @stlite/desktop also includes browserslist setting.
It also should be fixed like above, however, it can lead to larger bundle size.

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.

1 participant