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

Be explicit on the python dependencies #7

Closed
wants to merge 1 commit into from

Conversation

lu-zero
Copy link

@lu-zero lu-zero commented Jun 25, 2018

No description provided.

@glguy
Copy link
Contributor

glguy commented Jun 25, 2018

Would it make sense to create a requirements.txt to help automate this?

https://pip.pypa.io/en/stable/user_guide/#requirements-files

@lu-zero
Copy link
Author

lu-zero commented Jun 25, 2018

I guess all depends on how you'd consider eventually distributing the whole package.
Right now it has plenty of moving parts and I guess we are far from a final state.

A requirements.txt and mention of it in the README.md would be surely good for newcomers like me :)

@thedataking
Copy link
Contributor

@glguy yes, I added a requirements.txt file and updated the build scripts to use it on the internal branch. I'll update the README to mention it.

@thedataking
Copy link
Contributor

Closing issue. README.md has been updated to explain how to (mostly) automatically provision a build environment. Thanks for the suggestion @lu-zero!

@lu-zero lu-zero deleted the patch-1 branch June 27, 2018 08: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.

3 participants