Skip to content

Fix bug for startup under windows#144

Merged
tvst merged 1 commit intostreamlit:developfrom
felixnext:develop
Sep 19, 2019
Merged

Fix bug for startup under windows#144
tvst merged 1 commit intostreamlit:developfrom
felixnext:develop

Conversation

@felixnext
Copy link
Copy Markdown
Contributor

@felixnext felixnext commented Sep 19, 2019

Fixed bug that hinders the startup under windows by using platform specific SIGQUIT signal.

[x] By checking this box, I agree that all contributions to this project are made under the Apache 2.0 license.

@tvst
Copy link
Copy Markdown
Contributor

tvst commented Sep 19, 2019

Awesome, our first external contributor! Thanks for doing this 🎉

That said, we weren't expecting external contributors before launch (Oct 1st!), so we haven't set up our PR template accordingly. Can you add this to your PR to indicate that you agree with the Apache 2 license? (Lawyers be lawyering)

[x] By checking this box, I agree that all contributions to this project are made under the Apache 2.0 license.

Other than that, the PR looks good to me 👍

@tvst
Copy link
Copy Markdown
Contributor

tvst commented Sep 19, 2019

Thanks! I just triggered the e2e tests, and when they're done I'll go ahead and merge.

Thanks for contributing!!

@tvst tvst merged commit 670de52 into streamlit:develop Sep 19, 2019
tconkling added a commit to tconkling/streamlit that referenced this pull request Sep 23, 2019
* develop:
  Release 0.46.0 (streamlit#170)
  Magic fixes (streamlit#138)
  [docs] Add analytics; redirect /secret/docs; fix compilation problem (streamlit#149)
  Fix bug for startup under windows (streamlit#144)
  Responsive layout (streamlit#104)
  Add basic PR template
  Better method signatures (streamlit#88)
  Publish docs to both /docs and /secret/docs, until we deprecate /secret/docs in January. (streamlit#141)
  Rename/report2app (streamlit#126)
tconkling added a commit to tconkling/streamlit that referenced this pull request Sep 23, 2019
* develop: (54 commits)
  Removing uber demo from streamlit repo (streamlit#159)
  Release 0.46.0 (streamlit#170)
  Magic fixes (streamlit#138)
  [docs] Add analytics; redirect /secret/docs; fix compilation problem (streamlit#149)
  Fix bug for startup under windows (streamlit#144)
  Responsive layout (streamlit#104)
  Add basic PR template
  Better method signatures (streamlit#88)
  Publish docs to both /docs and /secret/docs, until we deprecate /secret/docs in January. (streamlit#141)
  Rename/report2app (streamlit#126)
  Test running streamlit commands "bare" (streamlit#133)
  Updates to LP and sidebar. (streamlit#134)
  tests for z-index of date input popover in sidebar (streamlit#131)
  cypress test for escaping html in markdown (streamlit#125)
  On a Mac, check if xcode installed before requiring watchdog (streamlit#91)
  [Docs] Fix st.slider API in tutorial (streamlit#98)
  Sidebar exceptions (streamlit#107)
  Fixing unbound local variable (streamlit#110)
  Support hashing dataframes with unhashable objects. Gracefully f… (streamlit#118)
  Fix hashing if the object has a name but the name is not a string. (streamlit#117)
  ...
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