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

Placeholders #26

Merged
merged 2 commits into from
Nov 2, 2022
Merged

Placeholders #26

merged 2 commits into from
Nov 2, 2022

Conversation

amandakube
Copy link
Collaborator

This PR is based on the conversations we had in our meeting last week. Dan wanted placeholder chapters before providing the link to students. I also reordered chapters based on the discussion we had last week.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@jesteria
Copy link
Member

jesteria commented Nov 2, 2022

Cool. So what's the plan for chapter 9, moved to 09_old?

@amandakube
Copy link
Collaborator Author

The plan is to incorporate that material into one of the other chapters so I didn't want to get rid of it yet, but eventually we will delete those files. We will be working on reincorporating that over the next week or so

Copy link
Member

@jesteria jesteria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool thanks 👍

@jesteria jesteria merged commit af84430 into master Nov 2, 2022
@jesteria jesteria deleted the placeholders branch November 2, 2022 18:39
SusannaLange pushed a commit to SusannaLange/textbook-datascience-1 that referenced this pull request Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content textbook content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants