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

add requirements file #10

Closed
wants to merge 3 commits into from
Closed

add requirements file #10

wants to merge 3 commits into from

Conversation

nhilton92
Copy link
Collaborator

@nhilton92 nhilton92 commented Jan 10, 2019

Requirements file so that anyone can get started with the repo by pip installing these packages into their virtual environment

I added the current modules for any package used in all files - Excluding standard python 3.6 packages

You can use pip install -r --upgrade requirements.txt from within the base directory to install these packages

Partly addresses issue in #6

NickHilton and others added 3 commits January 9, 2019 18:09
 
 Current modules used in all files
 For easy use with pip install -r --upgrade requirements.txt
@frhino frhino requested a review from pahdo January 11, 2019 06:51
@pahdo
Copy link
Collaborator

pahdo commented Jan 11, 2019

Looks good! Thanks for putting this together.

@pahdo
Copy link
Collaborator

pahdo commented Jan 11, 2019

Can you squash these commits when merging?

@VincentLa14
Copy link
Member

VincentLa14 commented Jan 12, 2019

@pahdo I've made it a setting to only allow squash commits when merging a pull request. I generally agree with this approach as it keeps master a bit cleaner.

Copy link
Member

@VincentLa14 VincentLa14 left a comment

Choose a reason for hiding this comment

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

Also pending conversation here: https://sfbrigade.slack.com/archives/CEL6C4Q49/p1547185246007700 I would say lets hold off merging a tiny bit as we resolve whether we should use Anaconda or pip/virtual env to manage our environments.

@VincentLa14
Copy link
Member

See #21

@nhilton92
Copy link
Collaborator Author

Happy to move over to Anaconda as we discussed

@nhilton92 nhilton92 closed this Jan 17, 2019
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.

4 participants