Skip to content
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

Update extras_require to include requirements for examples #1140

Merged
merged 6 commits into from Aug 14, 2020

Conversation

kitchoi
Copy link
Contributor

@kitchoi kitchoi commented Aug 13, 2020

This provides for point 2 of #1125
This PR updates the extras_requires to include dependencies for the demo examples.

We should mention this in the README, but then we should also mention installation options for PyQt5 etc... they aren't there at the moment, and I decided not to mix them in this PR.

@kitchoi
Copy link
Contributor Author

kitchoi commented Aug 13, 2020

For the record, pip install chaco on my Python3.8 environment is currently failing with some errors from compiling C++ code (that's actually for enable), but I think it is orthogonal to this: We still need to declare what TraitsUI's examples depend on.

Copy link
Contributor

@rahulporuri rahulporuri left a comment

Choose a reason for hiding this comment

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

LGTM

@kitchoi kitchoi merged commit e6163f9 into master Aug 14, 2020
@kitchoi kitchoi deleted the 1125-demo-requirements branch August 14, 2020 13:35
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.

None yet

2 participants