Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

chore: add docfx session to noxfile, normalize - #100

Merged
gcf-merge-on-green[bot] merged 2 commits into
masterfrom
chore-add-docfx-session
Aug 5, 2020
Merged

chore: add docfx session to noxfile, normalize#100
gcf-merge-on-green[bot] merged 2 commits into
masterfrom
chore-add-docfx-session

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Unbreaking the docs-presubmit build.

@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Aug 5, 2020
@tmatsuo

Copy link
Copy Markdown
Contributor

@tseaver Thanks, I think we should also run synthtool to have other changes, in order to see if the test is actually passing.

@tseaver

Copy link
Copy Markdown
ContributorAuthor

@tmatsuo I lost my synthtool-fu during my hiatus. Can I just get my local copy updated to current master and run? Are there new OS-level dependencies?

@tmatsuo

Copy link
Copy Markdown
Contributor

@tseaver Maybe I can add a commit to the branch.

For the record, I was told there is a dependency on bazel.

@tmatsuo

Copy link
Copy Markdown
Contributor

I think you rebased, that should work too. Thanks!

@tseaver

Copy link
Copy Markdown
ContributorAuthor

@tmatsuo I merged #99 to master, and then merged it here, which seems to be getting docs-presubmit to actually run.

@tmatsuo

Copy link
Copy Markdown
Contributor

The docs-presubmit is passing :)

Do you know why the samples are failing?

@tmatsuo

Copy link
Copy Markdown
Contributor

At a glance, the test has sth like time.sleep(10), so they might be eventually consistent tests.

@tseaver

Copy link
Copy Markdown
ContributorAuthor

@tmatsuo We are hitting quota issues, breaking snippets / systests / samples.

@tseavertseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 5, 2020
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 5, 2020
@tseavertseaver added the automerge Merge the pull request once unit tests and other checks pass. label Aug 5, 2020
@gcf-merge-on-green
gcf-merge-on-greenBot merged commit fda22ae into masterAug 5, 2020
@tseaver
tseaver deleted the chore-add-docfx-session branch August 5, 2020 20:39
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automergeMerge the pull request once unit tests and other checks pass.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@tmatsuo@yoshi-kokoro