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

Update index.rst #215

Closed
wants to merge 1 commit into from
Closed

Update index.rst #215

wants to merge 1 commit into from

Conversation

nsaphra
Copy link
Contributor

@nsaphra nsaphra commented Dec 14, 2016

No description provided.

@yoavg
Copy link
Contributor

yoavg commented Dec 14, 2016

@nsaphra @pmichel31415 hi, I am trying to figure out the relation between #211 #212 #215 .
#212 definitely goes in, but are #211 and #215 compatible / needed?

@pmichel31415
Copy link
Collaborator

I think #212 takes care of #211 and #215 by reorganizing the tutorials

@nsaphra
Copy link
Contributor Author

nsaphra commented Dec 14, 2016

@yoavg As far as I can tell, without #215, the changes I made in #210 won't be visible in navigation. Correct me if I'm wrong, but currently the python tutorials don't show up in the index or on the sidebar navigation.

I think #211 should be merged because adding the python tutorials as a separate file made the deleted python tutorial section redundant.

@nsaphra
Copy link
Contributor Author

nsaphra commented Dec 14, 2016

I see that #212 reverts back to including python in the main tutorial rather than in its own file and makes #211 and #210 unnecessary. I will close them.

@nsaphra nsaphra closed this Dec 14, 2016
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