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

Edit Python overview #169

Merged
merged 2 commits into from
Apr 6, 2017
Merged

Conversation

willingc
Copy link
Contributor

@willingc willingc commented Apr 6, 2017

Closes #143.

Addresses #81.

@willingc willingc changed the title Python setup Edit Python overview Apr 6, 2017
@tracykteal
Copy link
Contributor

Thanks, this is terrific! Besides having this in Setup, I think it would be good to have some of that where instructors knew to go through it during the workshop.

Maybe we can leave that PR as is, and either some of it can be added to 'intro to Python' at the very beginning, or it could be its own episode, and it could be the first one taught, so 00-intro-to-jupyter-notebook? (I'd just have to figure out what happens if you have more than one 00 lesson on ordering.)

@willingc
Copy link
Contributor Author

willingc commented Apr 6, 2017

Let's merge and then do a separate issue that reorganizes the way you would like it.

@wrightaprilm
Copy link
Contributor

I would prefer to merge this as is and do reorganization later, if need be. I'm catching up on PRs now - are all the lessons getting 00-level intros to the tools used (Jupyter notebook, rstudio, whatever SQL does)?

@wrightaprilm wrightaprilm merged commit 2329536 into datacarpentry:gh-pages Apr 6, 2017
@wrightaprilm
Copy link
Contributor

Reads well; great work. Merging now.

@willingc willingc deleted the python-setup branch April 7, 2017 01:05
@willingc
Copy link
Contributor Author

willingc commented Apr 7, 2017

Thanks @wrightaprilm. Happy to help.

maxim-belkin pushed a commit that referenced this pull request May 9, 2018
zkamvar pushed a commit that referenced this pull request May 8, 2023
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.

None yet

3 participants