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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update docs build (via synth) #58

Merged
merged 8 commits into from Aug 5, 2020
Merged

docs: update docs build (via synth) #58

merged 8 commits into from Aug 5, 2020

Conversation

yoshi-automation
Copy link
Contributor

This PR was generated using Autosynth. 馃寛

Synth log will be available here:
https://source.cloud.google.com/results/invocations/4be218b6-dac7-413a-9b70-c61c140820cb/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@bfcdbe0
Source-Link: googleapis/synthtool@39b527a
Source-Link: googleapis/synthtool@5dfda56
Source-Link: googleapis/synthtool@ee7506d
Source-Link: googleapis/synthtool@f07cb44

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.
* feat(python-library): changes to docs job

* migrate to Trampoline V2
* add docs-presubmit job
* create docfx yaml files and upload them to another bucket

* remove redundant envvars

Source-Author: Takashi Matsuo <tmatsuo@google.com>
Source-Date: Wed Jul 29 16:15:18 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: f07cb4446192952f19be3056957f56d180586055
Source-Link: googleapis/synthtool@f07cb44
This reverts commit f07cb4446192952f19be3056957f56d180586055.

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Wed Jul 29 17:33:57 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: ee7506d15daa3873accfff9430eff7e3953f0248
Source-Link: googleapis/synthtool@ee7506d
* feat(python-library): changes to docs job

* migrate to Trampoline V2
* add docs-presubmit job
* create docfx yaml files and upload them to another bucket

* remove redundant envvars

* add a failing test first

* fix TemplateSyntaxError: Missing end of comment tag

* serving_path is not needed any more

* use `raw` to make jinja happy

Source-Author: Takashi Matsuo <tmatsuo@google.com>
Source-Date: Thu Jul 30 12:44:02 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 5dfda5621df45b71b6e88544ebbb53b1a8c90214
Source-Link: googleapis/synthtool@5dfda56
Source-Author: Takashi Matsuo <tmatsuo@google.com>
Source-Date: Thu Jul 30 18:26:35 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 39b527a39f5cd56d4882b3874fc08eed4756cebe
Source-Link: googleapis/synthtool@39b527a
Source-Author: Takashi Matsuo <tmatsuo@google.com>
Source-Date: Fri Jul 31 16:17:13 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: bfcdbe0da977b2de6c1c0471bb6dc2f1e13bf669
Source-Link: googleapis/synthtool@bfcdbe0
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 5, 2020
@tseaver tseaver changed the title [CHANGE ME] Re-generated to pick up changes from synthtool. docs: update docs build (via synth) Aug 5, 2020
@tseaver
Copy link
Contributor

tseaver commented Aug 5, 2020

Blocked on PR #59.

@tseaver tseaver added the status: blocked Resolving the issue is dependent on other work. label Aug 5, 2020
@tmatsuo
Copy link
Contributor

tmatsuo commented Aug 5, 2020

I think the warning about conf.py is a red herring.

I also see an error in the log:

sphinx.errors.ExtensionError: source_parser for 'md' is already registered

@tmatsuo
Copy link
Contributor

tmatsuo commented Aug 5, 2020

I don't know, but it might be worth a try. I'll add a commit to the branch to see if the test actually passing.

@tmatsuo
Copy link
Contributor

tmatsuo commented Aug 5, 2020

Might be relevant:
readthedocs/recommonmark#146 (comment)

@tmatsuo
Copy link
Contributor

tmatsuo commented Aug 5, 2020

I compared the docs/conf.py to the same file in other library, and I have speculative changes now.

@tmatsuo
Copy link
Contributor

tmatsuo commented Aug 5, 2020

The builds are passing now. Merging.

@tmatsuo tmatsuo merged commit 5bdacd4 into master Aug 5, 2020
@tseaver tseaver deleted the autosynth-synthtool branch August 5, 2020 19:40
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. context: partial status: blocked Resolving the issue is dependent on other work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants