-
Notifications
You must be signed in to change notification settings - Fork 254
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
fixes for CI #2172
fixes for CI #2172
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
python build_source_docs.py | ||
jupyter-book build -W --keep-going openmdao_book | ||
python copy_build_artifacts.py | ||
./build_all_docs.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to make sure, have you verified that an error raised from the jupyter-book build will trigger an error on this workflow? I remember having troubles when some things were in shell scripts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah.. no, I was not aware of that wrinkle... I can back this change out...
Summary
networkx<2.6
for now until Compatibility with NetworkX 2.6 #2171 is addressedRelated Issues
Backwards incompatibilities
None
New Dependencies
None