-
Notifications
You must be signed in to change notification settings - Fork 629
feat: add fork filter parameter to repository search functionality #250
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
Thanks @juruen it is a pleasure. So the current search repository func can't search profile(so to say, or my repo) and return all repositories including forked ones. When users try to list their repos, it doesnt return forked ones. The query you have written is only searching a particular repo. |
Basically I am not able to list my forked repositories. |
Yes, I will probably try refine my queries again. Seems strange |
@MayorFaj closing this for now as we discussed an alternative using an existing tool. Feel free to re-open this or open an issue to discuss this further. But once again, thank so much for your help here! ❤️ |
This pull request introduces a new feature to filter repository search results by fork status. The changes include updates to the
README.md
, the main search function, and the test cases to support this new functionality.New feature for fork filtering:
README.md
: Added a description for the newfork
parameter in the search query options.Code updates for fork filtering:
pkg/github/search.go
: Imported thestrings
package and added thecontainsForkFilter
function to check for existing fork filters in the query. Updated theSearchRepositories
function to include thefork
parameter and modify the query accordingly. [1] [2]Test updates for fork filtering:
pkg/github/search_test.go
: Added new test cases to verify the behavior of thefork
parameter in repository searches, including scenarios where the parameter is not provided, set to 'only', 'true', or 'false'. [1] [2]