- Notifications
You must be signed in to change notification settings - Fork2.7k
Changed q to query in search#740
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR standardizes the parameter name fromq toquery for the search code functionality, completing the migration to more descriptive parameter naming across the codebase.
- Renamed parameter
qtoqueryin the search code tool implementation - Updated all corresponding test cases and assertions to use the new parameter name
- Updated documentation to reflect the parameter name change
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/github/search.go | Updated parameter definition and extraction fromq toquery |
| pkg/github/search_test.go | Updated test assertions and test data to usequery instead ofq |
| pkg/github/toolsnaps/search_code.snap | Updated snapshot test data to reflect the parameter name change |
| README.md | Updated documentation to showquery parameter instead ofq |
efef8ae intomainUh oh!
There was an error while loading.Please reload this page.
tim2346 commentedJul 25, 2025
Merged |
This it the last place where we're using
qinstead ofquery.