Skip to content

ci: fix docs ci - #401

Merged
gcf-merge-on-green[bot] merged 5 commits into
masterfrom
grant_docs_ci
Dec 2, 2021
Merged

gcf-merge-on-green[bot] merged 5 commits into
masterfrom
grant_docs_ci

Conversation

@grant

@grant grant commented Dec 2, 2021 •

Copy link
Copy Markdown
Contributor

It looks like the doc CI check would not produce the right exit code. This should correctly report bad doc errors (see below).

  • Fixes docs CI error code
  • Runs docs CI

Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
@grant grant self-assigned this Dec 2, 2021
@google-cla google-cla Bot added the cla: yes label Dec 2, 2021
@grant grant changed the title ci: fix docs ci - bad ci: fix docs ci Dec 2, 2021
grant added 2 commits December 1, 2021 21:31
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
@grant

grant commented Dec 2, 2021

Copy link
Copy Markdown
Contributor Author

Ironically, docs CI failed for a different PR appropriately.

#400

I suspect this CI must not fail for certain changes. Anyways, I believe this CI change is more stable.

@matthewrobertson matthewrobertson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
@grant grant added the automerge Summon MOG for automerging label Dec 2, 2021
@gcf-merge-on-green
gcf-merge-on-green Bot merged commit 2073558 into master Dec 2, 2021
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Summon MOG for automerging label Dec 2, 2021
@grant
grant deleted the grant_docs_ci branch December 3, 2021 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants