plotly / dash-enterprise-auth

Python Auth Client for Dash Enterprise
MIT License
4 stars 3 forks source link

Update requirements #8

Closed wbrgss closed 4 years ago

wbrgss commented 4 years ago

These removed requirements should be already included via Dash's setup.py via requires-install.txt

The setup.cfg was also updated to reflect those changes.

wbrgss commented 4 years ago

@antoinerg kind of a long shot but I wonder if the first line of requirements.txt (dash-core-components>=0.40.0) is messing up your aforementioned Python 2.7 Snapshots docker install somehow

antoinerg commented 4 years ago

@wbrgss You were on the right track. Do you mind if I push an extra commit here?

wbrgss commented 4 years ago

@antoinerg be my guest!

antoinerg commented 4 years ago

With https://github.com/plotly/dash-enterprise-auth/pull/8/commits/9b5b86a43d9087d3a94890b8307c635b0618ab7a the following command docker run -it -v $(pwd):$(pwd) -w $(pwd) python:2.7 pip install -e . now suceeds :tada:

antoinerg commented 4 years ago

This is ready for review @chriddyp @alexcjohnson !

chriddyp commented 4 years ago

💃