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

chore: export search funcs#167

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

Merged
SamMorrowDrums merged 6 commits intomainfromexport-search
Apr 8, 2025
Merged

chore: export search funcs#167

SamMorrowDrums merged 6 commits intomainfromexport-search
Apr 8, 2025

Conversation

SamMorrowDrums
Copy link
Collaborator

Export search funcs to public interface.

@CopilotCopilotAI review requested due to automatic review settingsApril 7, 2025 22:32
Copy link
Contributor

@CopilotCopilotAI left a 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 exports several GitHub search and resource functions to the public interface by renaming them from unexported (lowercase) to exported (PascalCase) versions. The changes update function names and comments throughout the code as well as the corresponding test references to ensure consistency.

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
pkg/github/server.goRenamed resource and tool functions to exported variants.
pkg/github/search_test.goUpdated test references for renamed search functions.
pkg/github/search.goExported search functions by renaming them.
pkg/github/repository_resource_test.goUpdated tests with exported resource handler functions.
pkg/github/repository_resource.goRenamed resource functions and their comments to be exported.
pkg/github/repositories_test.goUpdated test references for repository functions.
pkg/github/repositories.goExported repository functions by renaming them.
pkg/github/pullrequests_test.goUpdated test references for pull request functions.
pkg/github/pullrequests.goRenamed pull request functions to the exported naming convention.
pkg/github/issues_test.goUpdated test references for issue functions.
pkg/github/issues.goExported issue functions with updated naming and comments.
pkg/github/code_scanning_test.goUpdated test references for code scanning functions.
pkg/github/code_scanning.goRenamed code scanning functions to be exported.

@SamMorrowDrumsSamMorrowDrums changed the base branch frommain toexport-repositoriesApril 7, 2025 22:33
juruen
juruen previously approved these changesApr 8, 2025
Copy link
Collaborator

@juruenjuruen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

🚀

Base automatically changed fromexport-repositories tomainApril 8, 2025 07:34
@SamMorrowDrumsSamMorrowDrums dismissedjuruen’sstale reviewApril 8, 2025 07:34

The base branch was changed.

@SamMorrowDrumsSamMorrowDrums merged commit3dae8ab intomainApr 8, 2025
13 of 15 checks passed
@SamMorrowDrumsSamMorrowDrums deleted the export-search branchApril 8, 2025 07:39
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@juruenjuruenjuruen left review comments

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
@SamMorrowDrums@juruen

[8]ページ先頭

©2009-2025 Movatter.jp