Skip to content

Conversation

ArangoGutierrez
Copy link
Collaborator

@ArangoGutierrez ArangoGutierrez commented Sep 18, 2025

This aims to further reduce the diff in #1280

Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 17835281866

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 36.711%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/config/toml/source-empty.go 0 2 0.0%
Totals Coverage Status
Change from base Build 17833701500: 0.0%
Covered Lines: 4800
Relevant Lines: 13075

💛 - Coveralls

Copy link

@Copilot Copilot AI 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 makes the NewEmpty function public by changing its visibility from private (newEmpty) to public (NewEmpty) and updating the corresponding function call.

  • Renamed newEmpty function to NewEmpty to make it publicly accessible
  • Updated the function call in the Load method to use the new public function name

@ArangoGutierrez ArangoGutierrez marked this pull request as ready for review September 18, 2025 17:17
@elezar elezar merged commit f49c5dc into NVIDIA:main Sep 19, 2025
16 checks passed
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.

3 participants