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

Changes to Episode 2 #50

Open
wants to merge 3 commits into
base: gh-pages
Choose a base branch
from
Open

Changes to Episode 2 #50

wants to merge 3 commits into from

Conversation

c-martinez
Copy link
Collaborator

@c-martinez c-martinez commented Jun 16, 2017

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

Removed the section on variables persisting in Jupyter notebook cells, and moved a couple of things around.
@richyvk
Copy link
Collaborator

richyvk commented Jun 16, 2017

Reviewed and edited. Ready to merge IMO.

@c-martinez
Copy link
Collaborator Author

Thanks @richyvk !

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

@richyvk
Copy link
Collaborator

richyvk commented Jun 18, 2017

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
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants