Skip to content

docs: add guides for occ, configure_client, and troubleshooting#5441

Open
bshaffer wants to merge 12 commits intogoogleapis:mainfrom
bshaffer:add-product-neutral-guides
Open

docs: add guides for occ, configure_client, and troubleshooting#5441
bshaffer wants to merge 12 commits intogoogleapis:mainfrom
bshaffer:add-product-neutral-guides

Conversation

@bshaffer
Copy link
Copy Markdown

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.73%. Comparing base (85388cd) to head (1b3cca1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5441      +/-   ##
==========================================
- Coverage   97.74%   97.73%   -0.01%     
==========================================
  Files         218      218              
  Lines       48235    48235              
==========================================
- Hits        47145    47144       -1     
- Misses       1090     1091       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coryan
Copy link
Copy Markdown
Contributor

coryan commented Apr 16, 2026

The deny failures are probably because this PR needs to rebase to HEAD. You are hitting vulnerability errors that we already fixed.

The formatting error is more complicated:


Error: Could not format "/workspace/guide/src/occ.md".
--
  |  
  | The formatted Markdown renders to different HTML than the input Markdown. This
  | is likely a bug in mdformat. Please create an issue report here, including the
  | input Markdown: https://github.com/executablebooks/mdformat/issues


Maybe we need to update mdformat ??

Copy link
Copy Markdown
Member

@dbolduc dbolduc left a comment

Choose a reason for hiding this comment

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

I only looked at the proxy section

Comment thread guide/src/configure_client.md Outdated
Comment thread guide/src/configure_client.md Outdated
Copy link
Copy Markdown
Contributor

@coryan coryan left a comment

Choose a reason for hiding this comment

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

Just some drive-by comments.

Comment thread guide/src/occ.md Outdated
Comment thread guide/src/troubleshooting.md Outdated
Comment thread guide/src/configure_client.md Outdated
Comment thread guide/src/configure_client.md
Comment thread guide/src/configure_client.md Outdated
Comment thread guide/src/configure_client.md Outdated
Comment thread guide/src/configure_client.md Outdated
Comment thread ci/requirements.txt
@bshaffer bshaffer marked this pull request as ready for review April 21, 2026 16:01
@bshaffer bshaffer requested a review from a team as a code owner April 21, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants