-
Notifications
You must be signed in to change notification settings - Fork 156
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 unused variables in build_docs script #1469
remove unused variables in build_docs script #1469
Conversation
I realized that cuspatial/cpp/doxygen/Doxyfile Line 41 in e002dc8
cuspatial/cpp/doxygen/Doxyfile Line 2174 in e002dc8
Just pushed a commit restoring them. I think the small diff left in this PR is still worth it, to replace a hard-coded version number with one that's driven by the |
/merge |
Description
Contributes to rapidsai/build-planning#106
This project doesn't need any of the changes that are being made as part of that issue across most other repos... it's already doing all of its
pip
andconda
installs in CI in single installs (which is strict and safe) 🎉This small PR just makes its handling of environment variables in
ci/build_docs.sh
consistent with other RAPIDS repos, in the interest of keeping those scripts looking similar so it's easy to apply changes across all RAPIDS projects. ref: rapidsai/cudf#17013 (comment)Checklist