Skip to content

doc: use correct path in codegen command#14487

Merged
cuiy0006 merged 2 commits intogoogleapis:mainfrom
cuiy0006:use-new-path-generator-doc
Jul 17, 2024
Merged

doc: use correct path in codegen command#14487
cuiy0006 merged 2 commits intogoogleapis:mainfrom
cuiy0006:use-new-path-generator-doc

Conversation

@cuiy0006
Copy link
Copy Markdown
Contributor

@cuiy0006 cuiy0006 commented Jul 17, 2024

The old path will cause an exception,

terminate called after throwing an instance of 'nlohmann::json_abi_v3_11_3::detail::type_error'
  what():  [json.exception.type_error.305] cannot use operator[] with a string argument with null
Aborted

Replace with new path which we already have in generate-libraries.sh.


This change is Reviewable

@cuiy0006 cuiy0006 requested a review from a team July 17, 2024 21:05
@cuiy0006 cuiy0006 enabled auto-merge (squash) July 17, 2024 21:24
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.59%. Comparing base (695f254) to head (0d08100).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #14487   +/-   ##
=======================================
  Coverage   93.59%   93.59%           
=======================================
  Files        2316     2316           
  Lines      207354   207354           
=======================================
+ Hits       194080   194083    +3     
+ Misses      13274    13271    -3     

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

@cuiy0006 cuiy0006 merged commit 07d6fd7 into googleapis:main Jul 17, 2024
@cuiy0006 cuiy0006 deleted the use-new-path-generator-doc branch July 17, 2024 21:49
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.

2 participants