Skip to content

refactor: rename shadowed vars for clarity - #367

Closed
tenthirtyam wants to merge 1 commit into
hashicorp:mainfrom
tenthirtyam:refactor/rename-shadowed-vars
Closed

tenthirtyam wants to merge 1 commit into
hashicorp:mainfrom
tenthirtyam:refactor/rename-shadowed-vars

Conversation

@tenthirtyam

Copy link
Copy Markdown
Contributor

Description

This refactor renames several local variables that were shadowing imported package names or using ambiguous names (for example structtag, path, uuid, and errors).

It improves readability and avoids confusion in helper methods and tests without changing behavior.

Rollback Plan

Revert commit.

Changes to Security Controls

None.

@tenthirtyam
tenthirtyam requested a review from a team as a code owner September 9, 2026 20:15
This refactor renames several local variables that were shadowing imported package names or using ambiguous names (for example `structtag`, `path`, `uuid`, and `errors`). It improves readability and avoids confusion in helper methods and tests without changing behavior.

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