Skip to content

Rename invalidParams to validateToolParams#12

Merged
rakyll merged 1 commit intomainfrom
validate2
Apr 18, 2025
Merged

Rename invalidParams to validateToolParams#12
rakyll merged 1 commit intomainfrom
validate2

Conversation

@rakyll
Copy link
Copy Markdown
Contributor

@rakyll rakyll commented Apr 18, 2025

Methods should be verbs. Fixes #4.

@rakyll rakyll requested a review from NTaylorMullen April 18, 2025 00:09
Copy link
Copy Markdown
Collaborator

@NTaylorMullen NTaylorMullen left a comment

Choose a reason for hiding this comment

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

This may conflict with some of the auto fixing lint issues I just merged as an FYI

Methods should be verbs. Fixes #4.
@rakyll rakyll merged commit e1fac40 into main Apr 18, 2025
2 checks passed
@rakyll rakyll deleted the validate2 branch April 21, 2025 23:27
keithballinger pushed a commit that referenced this pull request Jun 13, 2025
yaleh pushed a commit to yaleh/gemini-cli that referenced this pull request Jun 29, 2025
viridivn pushed a commit to viridivn/gemini-cli that referenced this pull request Jun 29, 2025
yewenchen pushed a commit to yewenchen/gemini-cli that referenced this pull request Jul 8, 2025
Adib234 pushed a commit that referenced this pull request Nov 11, 2025
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.

Rename Tool.invalidParams to Tool.verifyParams

3 participants