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

Remove links to transforming data page #1280

Merged
merged 5 commits into from Apr 12, 2017

Conversation

Projects
None yet
2 participants
@jlstevens
Member

jlstevens commented Apr 12, 2017

This PR addresses #1077. We opted to remove links to the transforming data page as it was incomplete and not in the notebook format. In addition the links to it were mostly mentioning slicing so the Sampling Data tutorial seems more appropriate anyway.

I don't think there is much to discuss and unless I get some feedback, I'll just trigger a docs build and merge.

@jlstevens

This comment has been minimized.

Member

jlstevens commented Apr 12, 2017

I've rebuilt the docs on build.holoviews.org and fixed various links in the README. I think this PR is ready to merge.

@jlstevens jlstevens added the ready label Apr 12, 2017

@philippjfr

This comment has been minimized.

Member

philippjfr commented Apr 12, 2017

Thanks looks good.

@philippjfr philippjfr merged commit ee70c32 into master Apr 12, 2017

4 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 79.03%
Details
s3-reference-data-cache Test data is cached.
Details

@philippjfr philippjfr deleted the remove_transforming_data_references branch Apr 19, 2017

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