Skip to content

refactor: prefer reflect.Pointer over deprecated Ptr alias - #357

Closed
tenthirtyam wants to merge 1 commit into
hashicorp:mainfrom
tenthirtyam:refactor/prefer-reflect-pointer
Closed

tenthirtyam wants to merge 1 commit into
hashicorp:mainfrom
tenthirtyam:refactor/prefer-reflect-pointer

Conversation

@tenthirtyam

Copy link
Copy Markdown
Contributor

Description

reflect.Ptr is marked //go:fix inline to Pointer.

Use the canonical Kind name in the test helper switch.

Resolved Issues

reflect.Ptr is marked //go:fix inline to Pointer.

Rollback Plan

Revert commit.

Changes to Security Controls

None.

@tenthirtyam
tenthirtyam requested a review from a team as a code owner August 4, 2026 00:02
@tenthirtyam
tenthirtyam force-pushed the refactor/prefer-reflect-pointer branch from 24cb6ae to 2794bb4 Compare August 12, 2026 18:09
@tenthirtyam
tenthirtyam force-pushed the refactor/prefer-reflect-pointer branch from 2794bb4 to c34f201 Compare September 9, 2026 13:49
@tenthirtyam

Copy link
Copy Markdown
Contributor Author

@tanmay-hc - rebased and ready for review.

`reflect.Ptr` is marked `//go:fix inline` to `Pointer`; use the
canonical Kind name in the test helper switch.

Signed-off-by: Ryan Johnson <ryan@tenthirtyam.org>
@tenthirtyam

Copy link
Copy Markdown
Contributor Author

Superseded by the consolidated cleanup PR: #377. Closing this in favor of that PR per maintainer request.

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.

1 participant