Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Main#1572

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

Open
mudlook wants to merge1 commit intogithub:main
base:main
Choose a base branch
Loading
frommudlook:main
Open

Main#1572

mudlook wants to merge1 commit intogithub:mainfrommudlook:main

Conversation

@mudlook
Copy link

  • Enhance search functionality with new query filters and utility functions
  • Added multiple test cases for searching issues with various query filters inissues_test.go.
  • Introduced utility functions insearch_utils.go to check for specific filters and extract repository information from queries.
  • Created comprehensive tests insearch_utils_test.go to validate the new filtering logic and ensure accurate query parsing.
  • Refactor repository filter extraction in search utilities
  • RenamedextractRepoFilter tohasRepoFilter to simplify the function's purpose.
  • Updated test cases insearch_utils_test.go to reflect the new function name and logic.
  • Adjusted tests to focus on the presence of therepo: filter rather than extracting owner and repo details.
  • Enhance search functionality with additional query filters and utility functions
  • Added new test cases for searching pull requests and users with various query filters inpullrequests_test.go andsearch_test.go.
  • Implemented a utility functionhasTypeFilter insearch_utils.go to check for the presence oftype: filters in queries.
  • Updated the search handler to conditionally prepend thetype: filter based on the presence of existing filters, improving query handling.
  • Updated both regex patterns to handle this edge case

Closes:

* Enhance search functionality with new query filters and utility functions- Added multiple test cases for searching issues with various query filters in `issues_test.go`.- Introduced utility functions in `search_utils.go` to check for specific filters and extract repository information from queries.- Created comprehensive tests in `search_utils_test.go` to validate the new filtering logic and ensure accurate query parsing.* Refactor repository filter extraction in search utilities- Renamed `extractRepoFilter` to `hasRepoFilter` to simplify the function's purpose.- Updated test cases in `search_utils_test.go` to reflect the new function name and logic.- Adjusted tests to focus on the presence of the `repo:` filter rather than extracting owner and repo details.* Enhance search functionality with additional query filters and utility functions- Added new test cases for searching pull requests and users with various query filters in `pullrequests_test.go` and `search_test.go`.- Implemented a utility function `hasTypeFilter` in `search_utils.go` to check for the presence of `type:` filters in queries.- Updated the search handler to conditionally prepend the `type:` filter based on the presence of existing filters, improving query handling.* Updated both regex patterns to handle this edge case<!--    Thank you for contributing to GitHub MCP Server!    Please reference an existing issue: `Closes #NUMBER`    Screenshots or videos of changed behavior is incredibly helpful and always appreciated.    Consider addressing the following:    - Tradeoffs: List tradeoffs you made to take on or pay down tech debt.    - Alternatives: Describe alternative approaches you considered and why you discarded them.-->Closes:
@mudlookmudlook requested a review froma team as acode ownerDecember 11, 2025 09:51
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mudlook@Cbnc-Mk

[8]ページ先頭

©2009-2025 Movatter.jp