Skip to content

test: Add to package with Ptr function#1653

Merged
elezar merged 1 commit intoNVIDIA:mainfrom
elezar:add-as-package-for-conversion
Feb 17, 2026
Merged

test: Add to package with Ptr function#1653
elezar merged 1 commit intoNVIDIA:mainfrom
elezar:add-as-package-for-conversion

Conversation

@elezar
Copy link
Member

@elezar elezar commented Feb 17, 2026

Instead of duplicating the ptr function for getting the pointer to a constant in testing, this change adds a to package with a Ptr function and uses this in the tests.

Inspired by https://youtu.be/RZe8ojn7goo?si=eEFTd_jypfRIcS8W&t=1323

@elezar elezar requested review from jgehrcke and klueska February 17, 2026 08:00
Instead of duplicating the ptr function for getting the
pointer to a constant in testing, this change adds a to package
with a Ptr function and uses this in the tests.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the add-as-package-for-conversion branch from 22d9e18 to f2e1f16 Compare February 17, 2026 08:01
@elezar elezar merged commit 2e4e8c8 into NVIDIA:main Feb 17, 2026
9 of 10 checks passed
@coveralls
Copy link

coveralls commented Feb 17, 2026

Pull Request Test Coverage Report for Build 22090447996

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 3 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.008%) to 39.181%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/test/to/to.go 0 3 0.0%
Totals Coverage Status
Change from base Build 22067568877: -0.008%
Covered Lines: 5711
Relevant Lines: 14576

💛 - Coveralls

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