- Notifications
You must be signed in to change notification settings - Fork897
chore: export repository funcs#166
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 updates the naming of various repository functions and tool definitions to export them on the public interface.
- Renamed resource functions from lower-case to exported (capitalized) versions in pkg/github/server.go, pkg/github/repository_resource.go, and corresponding test files.
- Updated tool functions in pkg/github/repositories.go and pkg/github/repositories_test.go to use capitalized, exported names.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/server.go | Updated calls to resource functions to their exported versions. |
| pkg/github/repository_resource_test.go | Updated tests to use exported resource handler functions. |
| pkg/github/repository_resource.go | Renamed resource functions and handler to exported names. |
| pkg/github/repositories_test.go | Updated tests to use exported tool function names. |
| pkg/github/repositories.go | Renamed tool functions to exported names. |
Comments suppressed due to low confidence (1)
pkg/github/server.go:59
- [nitpick] The function 'searchRepositories' remains unexported while other repository functions have been updated to exported versions. Consider renaming it to 'SearchRepositories' for naming consistency if it is meant to be part of the public interface.
s.AddTool(searchRepositories(client, t))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.
🚀
7ec5938 tof2bbc19Comparec1bdd6a intomainUh oh!
There was an error while loading.Please reload this page.
Export repository funcs to the public interface.