Skip to content

Simplify API function names#2

Merged
Kansuler merged 1 commit intomainfrom
cleanup/simplify-interface
Oct 1, 2025
Merged

Simplify API function names#2
Kansuler merged 1 commit intomainfrom
cleanup/simplify-interface

Conversation

@Kansuler
Copy link
Copy Markdown
Owner

@Kansuler Kansuler commented Oct 1, 2025

Cleaned up the public API to be more concise and user-friendly:

  • Renamed NewConfigImpl()New()
  • Renamed LoadEnvironment()Load()
  • Renamed ConfigurationKeysRegistered()Exists()
  • Made internal ConfigImpl type private (config)
  • Updated all examples and documentation to reflect the new naming

The functionality remains the same, just with shorter and cleaner method names that follow Go conventions better.

Copilot AI review requested due to automatic review settings October 1, 2025 19:28
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 1, 2025

Golang Build & Test status 🚀

Go Test Outcome 🧪success

Coverage Report 📊success

Pusher: @Kansuler, Action: pull_request, Working Directory: ``, Workflow: Go Test Coverage Report On Pull Request

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Oct 1, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% (target: -1.00%) 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (84dd334) 446 403 90.36%
Head commit (a82ece7) 446 (+0) 403 (+0) 90.36% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#2) 28 28 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

This comment was marked as resolved.

@Kansuler Kansuler force-pushed the cleanup/simplify-interface branch from bac34e8 to a82ece7 Compare October 1, 2025 19:34
@Kansuler Kansuler requested a review from Copilot October 1, 2025 19:34

This comment was marked as resolved.

@Kansuler Kansuler merged commit 3f60272 into main Oct 1, 2025
4 checks passed
@Kansuler Kansuler deleted the cleanup/simplify-interface branch October 1, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants