dev/upgrade streamlit 1.27.0 #671
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
whitphx
commented
Jan 7, 2024
whitphx
commented
- Copy .nvmrc from [email protected]
- Update the streamlit submodule, fix some package resolutions for it, and fix @stlite/kernel for streamlit 1.27.0
- Update @stlite/common-react and mountable
- Set webpack version in @stlite/mountable as same as [email protected]
- Set the resolved webpack version to avoid an error "The 'compilation' argument must be an instance of Compilation", which is caused by multiple versions of webpack. Ref: https://qiita.com/Hanpen3019/items/e6ae5d29b334bc3d7af3
- Update @stlire/sharing
- Update @stlite/desktop
- Update Makefile
- Update the streamlit submodule
…and fix @stlite/kernel for streamlit 1.27.0
… argument must be an instance of Compilation", which is caused by multiple versions of webpack. Ref: https://qiita.com/Hanpen3019/items/e6ae5d29b334bc3d7af3
whitphx
force-pushed
the
dev/upgrade-streamlit-1.27.0
branch
from
b73a964
to
13769ab
Compare
whitphx
force-pushed
the
dev/upgrade-streamlit-1.27.0
branch
4 times, most recently
from
58ed378
to
cb05ac1
Compare
whitphx
force-pushed
the
dev/upgrade-streamlit-1.27.0
branch
from
5cf0652
to
ad9d15d
Compare
whitphx
commented
Jan 8, 2024
| @@ -3,7 +3,9 @@ | |||
| "name": "stlite", | |||
| "workspaces": [ | |||
| "packages/*", | |||
| "streamlit/frontend" | |||
| "streamlit/frontend", | |||
There was a problem hiding this comment.
NOTE: streamlit/frontend is still needed here because the streamlit-proto make rule calls yarn pbjs in this directory, and yarn run needs to be run in a workspace directory to find the command binary location.
…ream Streamlit frontend to avoid type errors
|
At this point, I could make |
* Add stlite modifications * Add back connection manager * Move methods * Some updates * Remove header * More cleanup * More changes * More cleanup * More modifications * Don't update pyiodide * More cleanup * Don't Update pyodide * Don't update pyiodide * More cleanup * More cleanup * Rename of option * Remove new line * Minor cleanup * Fix wrong option name * Minor renaming * Fix typo Co-authored-by: Yuichiro Tachibana (Tsuchiya) <[email protected]> * Update comment Co-authored-by: Yuichiro Tachibana (Tsuchiya) <[email protected]> * Remove paths * Change to put * Update URL * Add gatherUsageStats flag --------- Co-authored-by: Yuichiro Tachibana (Tsuchiya) <[email protected]>
github-actions
bot
requested a deployment
to
stlite-sharing (Preview)
|
Released as @lukasmasuch Thank you for your contribution! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.