-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
remove strong testing dependencies. #808
Conversation
percy | ||
requests[security] | ||
beautifulsoup4 | ||
waitress |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does testing
require
dash_flow_example==0.0.5
dash-dangerously-set-inner-html
virtualenv;python_version=="2.7"
mock
if not in the ci
environment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
they should go -ci.txt as only our integration tests need them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me double check the mock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💃
…down-#808 sets autoComplete to off in Dropdown plotly#808
sets autoComplete to off in Dropdown #808
this fixes #804
the separation could be easily handled by the extra_require. and the new
setup.py
.the only change might have an impact is for dash.testing users, they need to do
pip install dash[testing]
instead ofpip install dash
.