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

User guide 01 and 02 fixups #1690

Merged
merged 6 commits into from Jul 7, 2017

Conversation

Projects
None yet
2 participants
@ceball
Member

ceball commented Jul 6, 2017

First pass through 01 and 02.

Note that I have changed some example code, and I have potentially changed the meanings of some sentences. But no reorganization.

ceball added some commits Jul 6, 2017

First pass through 01 and 02; work in progress.
Also, will need to strip out various metadata changes :(
@jlstevens

This comment has been minimized.

Member

jlstevens commented Jul 6, 2017

Skimming the diffs, this all looks good. I think I'll probably have a look using nbdime diff to make sure all the meanings are still correct.

@jlstevens

This comment has been minimized.

Member

jlstevens commented Jul 7, 2017

Thanks for doing this! Changes look good.

@jlstevens jlstevens merged commit a3e669d into master Jul 7, 2017

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
s3-reference-data-cache Test data is cached.
Details

@philippjfr philippjfr deleted the user_guide_fixups branch Jul 31, 2017

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