FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Package docs files for simple deployment by daspecster · Pull Request #2033 · googleapis/google-cloud-python · GitHub

Package docs files for simple deployment - #2033

Merged
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:docs-updates
Jul 29, 2016
Merged

Package docs files for simple deployment#2033
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:docs-updates

Conversation

Copy link
Copy Markdown
Contributor
  • Working towards cleaner packaging for releases.
  • Fixed a bug in the navigation.

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 29, 2016

dhermes commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

Does this break the standard Sphinx docs, e.g. tox -e docs-rtd?

Copy link
Copy Markdown
Contributor Author

I don't think so, however it looks like it's broken anyway.

AFAICT there are no overlapping directories.
tox -e docs-rtd's output directories are sphinx-build -W -b html -d docs/_build_rtd/doctrees docs docs/_build_rtd/html

However I do get this error when running tox -e docs-rtd

- gcloud.error_reporting.client
ERROR: InvocationError: '/Users/daspecster/Documents/google/gcloud-py/.tox/docs-rtd/bin/python /Users/daspecster/Documents/google/gcloud-py/scripts/verify_included_modules.py'

I'll fix this real quick an make another PR.

dhermes commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

Does that mean tox -e docs is broken as well?

Copy link
Copy Markdown
Contributor Author

Oddly, tox -e docs seems to work.

dhermes commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

That is odd

Copy link
Copy Markdown
Contributor Author

Ok, so I uninstalled gcloud from my venv and reinstalled it via python setup.py develop and now tox -e docs-rtd and tox -e docs are both working again.

dhermes commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

tox should be isolated from your venv, that is the whole point of tox

Copy link
Copy Markdown
Contributor Author

Yeah, I'm really not sure what was going on at this point.

Maybe I had something funky going on and somehow it was picking up an old version of gcloud?

When your test commands execute they will execute with a PYTHONPATH setting that will lead Python to also import from the subdir below the directory where your tox.ini file resides.

dhermes commented Jul 29, 2016
edited
Loading

Copy link
Copy Markdown
Contributor

This LGTM provided it doesn't break RTD. Can you check when the new build happens

https://readthedocs.org/projects/gcloud-python/builds/

and make sure

http://gcloud-python.readthedocs.io/en/latest/

looks fine?

(UPDATE: For the record, RTD's build can't use tox -e docs, I just wanted to make sure that nothing was out of place.)

Copy link
Copy Markdown
Contributor Author

Will do!

daspecster merged commit bbf5053 into googleapis:master Jul 29, 2016
daspecster deleted the docs-updates branch July 29, 2016 18:35

Copy link
Copy Markdown
Contributor Author

Looks like ReadTheDocs built successfully. Clicking around on the site works good.

https://readthedocs.org/projects/gcloud-python/builds/4249133/

This was referenced Aug 3, 2016
parthea pushed a commit that referenced this pull request Mar 6, 2026
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Apr 1, 2026
* chore: update benchmark metrics

* fix metric calculation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL