feat(sidekick/rust): Generate samples for undelete methods.#3706
feat(sidekick/rust): Generate samples for undelete methods.#3706amanda-tarafa merged 1 commit intogoogleapis:mainfrom
Conversation
Undelete methods are identified as per AIP/135
Summary of ChangesHello @amanda-tarafa, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly enhances the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request adds support for generating samples for AIP-135 undelete methods in Rust. The changes introduce the necessary logic in the Go API model to identify these methods and update the Rust-specific mustache templates accordingly. The implementation is consistent with the existing support for other standard AIP methods like Get and Delete, and the new logic is well-tested. I have one suggestion to improve maintainability by reducing duplication in the templates, which could be addressed in a follow-up.
internal/sidekick/rust/templates/common/client_method_samples/builder_fields.mustache
Show resolved
Hide resolved
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3706 +/- ##
=======================================
Coverage 82.28% 82.29%
=======================================
Files 138 138
Lines 12713 12728 +15
=======================================
+ Hits 10461 10474 +13
- Misses 1774 1775 +1
- Partials 478 479 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This is ready for review, generation results can be seen in googleapis/google-cloud-rust#4340. |
Undelete methods are identified as per AIP/135