chore: export remaining + helpers #168
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR updates several functions and helpers to use exported (capitalized) names so they can be accessed externally. Key changes include renaming search functions and helper methods across multiple GitHub-related packages, and updating associated tests and resource handlers to reflect the new names.
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/server_test.go | Renamed getMe to GetMe in test cases. |
| pkg/github/server.go | Updated tool and helper function names to exported names. |
| pkg/github/search_test.go | Updated search function names to their exported versions. |
| pkg/github/search.go | Renamed search functions and updated with exported WithPagination. |
| pkg/github/repository_resource_test.go | Renamed resource handler functions to exported names. |
| pkg/github/repository_resource.go | Updated repository resource functions to exported names. |
| pkg/github/repositories_test.go | Renamed file-related functions to exported names in tests. |
| pkg/github/repositories.go | Updated repository tool functions with exported naming conventions. |
| pkg/github/pullrequests_test.go | Renamed pull request functions to exported names in tests. |
| pkg/github/pullrequests.go | Updated pull request functions to their exported counterparts. |
| pkg/github/issues_test.go | Renamed issue-related functions to exported names in tests. |
| pkg/github/issues.go | Updated issue functions to exported names. |
| pkg/github/code_scanning_test.go | Renamed code scanning alert functions to exported names in tests. |
| pkg/github/code_scanning.go | Updated code scanning functions to exported names. |
SamMorrowDrums
changed the title
SamMorrowDrums
force-pushed
the
export-rest
branch
from
e994049 to
dac50b2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Export the remaining tool funcs and helpers.