Skip to content

feat: add setup and test agentic skills - #321

Open
brianquinlan wants to merge 6 commits into
googleapis:mainfrom
brianquinlan:ready-release
Open

feat: add setup and test agentic skills #321
brianquinlan wants to merge 6 commits into
googleapis:mainfrom
brianquinlan:ready-release

Conversation

@brianquinlan

@brianquinlan brianquinlan commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Adds the new file generated/RELEASING.md, which describes the first steps in the generated-code release process. I will add more steps to that process as I complete them.

Restructures generated/README.md to avoid duplication with generated/RELEASING.md.

Modifies some test file to avoid duplicate imports now that every google_cloud_* library exports package:google_cloud_rpc/exceptions.dart.

The remaining changes are done through automation by completing steps 1-4 from generated/RELEASING.md.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation and configuration for generating and releasing Google Cloud API packages in Dart. It introduces a detailed release guide (RELEASING.md), updates the main README.md, and adds setup and testing markdown files (SKILL.md) for multiple Google Cloud services, alongside updating API source commits in librarian.yaml. The review feedback is highly constructive, pointing out a missing step in the release guide regarding updating the Librarian version in librarian.yaml, as well as minor grammatical and phrasing inconsistencies in the newly added setup and test markdown files.

Comment thread generated/RELEASING.md Outdated
@brianquinlan

Copy link
Copy Markdown
Contributor Author

/gcbrun

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