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

Remove colab vendor specific tags which cause nbdev_build_docs to fail #207

Merged
merged 1 commit into from
Aug 10, 2020

Conversation

butchland
Copy link

This adds an additional routine to find and remove the colab specific vendor tags which causes the nbdev_build_lib to fail.

Also modified clean notebook tests to add condition for additional colab specific vendor tags being removed by nbdev_clean_nbs command.

Fixes #206

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

👋 @butchland! Thank you for opening your first pull request in this repo. We are so happy that you have decided to contribute and value your contribution. Please read these materials before proceeding: Contributing Guide and Code of Conduct.

@butchland
Copy link
Author

weird error on build. nbdev_test_nbs fails on 00_export.ipynb even though I didn't modify it.
dunno how to fix. -- additional note: nbdev_test_nbs also fails on fastai master branch.

@jph00 jph00 closed this Aug 10, 2020
@jph00 jph00 reopened this Aug 10, 2020
@jph00
Copy link
Member

jph00 commented Aug 10, 2020

Sorry @butchland we've been fiddling around a bit with CI testing so we still have some failures. I'm re-running these tests now.

@jph00 jph00 merged commit 5d2d0cf into fastai:master Aug 10, 2020
@butchland butchland deleted the fix-vnd-colab-json-error branch August 11, 2020 00:49
@jph00 jph00 added the enhancement New feature or request label Sep 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nbdev_build_docs fails on colab specific vendor tags inside jupyter notebook cells
2 participants