Skip to content

Upgrade ProtobufJS and fix build script#2118

Merged
kmcgrady merged 1 commit intostreamlit:developfrom
kmcgrady:protobuf-upgrade
Oct 8, 2020
Merged

Upgrade ProtobufJS and fix build script#2118
kmcgrady merged 1 commit intostreamlit:developfrom
kmcgrady:protobuf-upgrade

Conversation

@kmcgrady
Copy link
Copy Markdown
Collaborator

@kmcgrady kmcgrady commented Oct 7, 2020

Due to a change in ProtobufJS, we have to change the way we wrap ES6 modules
to fix compilation. The commit that caused the issue is:

protobufjs/protobuf.js@6698d59

Due to a change in ProtobufJS, we have to change the way we wrap ES6 modules
to fix compilation. The commit that caused the issue is:

protobufjs/protobuf.js@6698d59
@kmcgrady kmcgrady requested review from a team and kantuni October 7, 2020 21:42
Copy link
Copy Markdown
Contributor

@akrolsmir akrolsmir left a comment

Choose a reason for hiding this comment

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

Woah, nice catch!

Copy link
Copy Markdown
Collaborator

@kantuni kantuni left a comment

Choose a reason for hiding this comment

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

LGTM!

@kmcgrady kmcgrady merged commit 8b674e2 into streamlit:develop Oct 8, 2020
tconkling added a commit to tconkling/streamlit that referenced this pull request Oct 12, 2020
# By karrie (7) and others
# Via GitHub
* develop:
  Removing cache option from main menu if s4a (streamlit#2149)
  Fix empty deploy page (streamlit#2148)
  Don't wait for unit tests before starting Cypress (streamlit#2142)
  Fix formatting of st.file_uploader docstring (streamlit#2141)
  Fix broken link in 0.68 changelog (streamlit#2144)
  Fix useEffect warning (streamlit#2137)
  Add global GTM container (streamlit#2128)
  Allow Streamlit server to handle Range Requests (streamlit#1967)
  rename hosted to hostedAt (streamlit#2132)
  Update change log
  Update notices
  Up version to 0.68.0
  Rename hosted to hostedAt in tracking data (streamlit#2132)
  Inject tracking data (streamlit#2110)
  [Feature Branch] File uploader (streamlit#2130)
  links for docs (streamlit#2129)
  Upgrade ProtobufJS and fix build script (streamlit#2118)
  Refresh landing page (streamlit#2116)
  Improve docstrings + tutorials for Layout (streamlit#2117)
  Better 'streamlit run' error message when no extension provided (streamlit#2115)

# Conflicts:
#	frontend/src/components/elements/Video/Video.tsx
#	frontend/src/components/widgets/FileUploader/FileUploader.test.tsx
#	frontend/src/components/widgets/FileUploader/FileUploader.tsx
#	frontend/src/lib/utils.ts
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.

3 participants