Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use value.TypeString in PathStep.String #306

Merged
merged 2 commits into from Aug 30, 2022
Merged

Conversation

dsnet
Copy link
Collaborator

@dsnet dsnet commented Jul 20, 2022

The value.TypeString function is what the rest of the package uses
and is slightly cleaner than using reflect.Type.String.

Updates #305

@dsnet dsnet requested a review from neild July 20, 2022 18:35
The value.TypeString function is what the rest of the package uses
and is slightly cleaner than using reflect.Type.String.

Updates #305
@dsnet dsnet changed the title Use value.TypeString in TypeAssertion.String Use value.TypeString in PathStep.String Aug 8, 2022
neild
neild approved these changes Aug 30, 2022
@neild neild merged commit 6606d4d into master Aug 30, 2022
25 checks passed
@dsnet dsnet deleted the dsnet/path-type-format branch September 3, 2022 06:29
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.

None yet

2 participants