Skip to content

[pull] main from googleapis:main#221

Merged
pull[bot] merged 3 commits intoCrazyForks:mainfrom
googleapis:main
Mar 17, 2026
Merged

[pull] main from googleapis:main#221
pull[bot] merged 3 commits intoCrazyForks:mainfrom
googleapis:main

Conversation

@pull
Copy link

@pull pull bot commented Mar 17, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

rahulpinto19 and others added 3 commits March 17, 2026 17:11
Change the issue labels for high visibility and quick fixing of the
broken links

#2669 (comment)
)

## Description

This PR improves the error handling and validation for the
`skills-generate` command and cleans up the unit test execution logs.

## Key Changes:
- **Positional Arguments**: Implemented `Args: cobra.NoArgs` to strictly
reject any unexpected positional arguments.
- **Test Output Cleanup**: Updated the test helper in command_test.go to
capture and silence Cobra's usage and error messages during test runs,
preventing them from polluting the terminal output.

## PR Checklist

> Thank you for opening a Pull Request! Before submitting your PR, there
are a
> few things you can do to make sure it goes smoothly:

- [ ] Make sure you reviewed

[CONTRIBUTING.md](https://github.com/googleapis/genai-toolbox/blob/main/CONTRIBUTING.md)
- [ ] Make sure to open an issue as a

[bug/issue](https://github.com/googleapis/genai-toolbox/issues/new/choose)
  before writing your code! That way we can discuss the change, evaluate
  designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)
- [ ] Make sure to add `!` if this involve a breaking change

🛠️ Fixes #<issue_number_goes_here>
@pull pull bot locked and limited conversation to collaborators Mar 17, 2026
@pull pull bot added the ⤵️ pull label Mar 17, 2026
@pull pull bot merged commit 1528d7c into CrazyForks:main Mar 17, 2026
4 of 10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants