search

thedataincubator / forecaster

3 stars 1 forks source link

Locking down conda packages #14

Open jonathan-lai-the-science-guy opened 6 years ago

jonathan-lai-the-science-guy commented 6 years ago

1) Locked down packages for conda-requirement.txt to the following specific versions;

pandas==0.23.1 bokeh==0.13.0 dill==0.2.8.2 flask==1.0.2 jinja2==2.10 pip==10.0.1 simplejson==3.15.0 nose==1.3.7 responses==0.9.0 mock==2.0.0

2) Added bash script grabVersion.sh to help speed up adding newer libraries to the conda-requirements.txt.

ZachGlassman commented 6 years ago

have a look at https://github.com/thedataincubator/forecaster/pull/13. Please either merge or rebase on this branch.

ChristinaZXY commented 6 years ago

Hi Zach,

I am a little bit confused about what to do. I have reviewed the code on

13 and selected approval. Does that suffice? Do I need to git clone the

repo on my local computer and merge/rebase the branch from there and then push it back to github?

Best, Xiaoyu

On Mon, Jul 2, 2018 at 12:29 PM, ZachGlassman notifications@github.com wrote:

have a look at #13 https://github.com/thedataincubator/forecaster/pull/13. Please either merge or rebase on this branch.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/thedataincubator/forecaster/pull/14#issuecomment-401861034, or mute the thread https://github.com/notifications/unsubscribe-auth/AkqVE29_qO3u06PYkDZ5o9Rj1h3pZzdFks5uCkprgaJpZM4U_jOD .

ZachGlassman commented 6 years ago

@ChristinaZXY You just need to review the code on this branch. @jonathan-lai-the-science-guy will need to merge the tests into this branch (or write new ones)