Skip to content

refactor: exported functions with unexported return types#319

Merged
tenthirtyam merged 1 commit intomainfrom
refactor/export-return-types
Jun 5, 2025
Merged

refactor: exported functions with unexported return types#319
tenthirtyam merged 1 commit intomainfrom
refactor/export-return-types

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

Export the return type for exported functions.

Renamed the configDeclaration type and its associated methods and variables to ConfigDeclaration. This aligns the naming convention with Go's standard practice of using PascalCase for exported types and methods.

Export the return type for exported functions.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam added this to the v1.2.0 milestone May 23, 2025
@tenthirtyam tenthirtyam self-assigned this May 23, 2025
@tenthirtyam tenthirtyam requested a review from a team as a code owner May 23, 2025 20:04
Copy link
Copy Markdown
Contributor

@kp2099 kp2099 left a comment

Choose a reason for hiding this comment

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

LGTM

@tenthirtyam tenthirtyam merged commit 206aa72 into main Jun 5, 2025
13 checks passed
@tenthirtyam tenthirtyam deleted the refactor/export-return-types branch June 5, 2025 14:28
@tenthirtyam tenthirtyam removed the request for review from JenGoldstrich June 5, 2025 14:28
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants