search

data-lessons / library-python-intro-DEPRECATED

REPO NOW MOVED > https://github.com/LibraryCarpentry/lc-python-intro
https://github.com/LibraryCarpentry/lc-python-intro
Other
1 stars 11 forks source link

Changes to Episode 2 #50

Open c-martinez opened 7 years ago

c-martinez commented 7 years ago

Worked on this during mozsprint, but never merged it. Never too late!

richyvk commented 7 years ago

Reviewed and edited. Ready to merge IMO.

c-martinez commented 7 years ago

Thanks @richyvk !

Added a couple more changes addressing issues mentioned by @elliewix in #35.

richyvk commented 7 years ago

Something gone a bit wrong here Carlos. At least one of the changes I made has been lost - removing the Variables Persist Between Cells section - line 108-115

I made the changes on a new commit on this branch - I'm wondering if you didn't update your copy before making your other changes??

Also, there is a issue with the repo when you clone it you get this error: warning: remote HEAD refers to nonexistent ref, unable to checkout.

Perhaps we can discuss on the Zoom?

c-martinez commented 7 years ago

It is very much likely that I did not pull changes before sending my new commit. Yes, let's fix this on zoom.