-
Notifications
You must be signed in to change notification settings - Fork 121
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
fix bug and delete String of Int and Float #205
Merged
Merged
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
a059685
fix bug and delete String of Int and Float
pjmd89 92a9c16
fix bug and delete String of Int and Float
pjmd89 899fdf3
Merge commit '92a9c169f8922811d1615d0dc51abf6b5fcf734c'
pjmd89 d35ffe4
Merge commit '899fdf351670d592534895bb1bfde9b4f67f5f64' into fixbug/v…
pjmd89 20a52d0
Merge branch 'vektah:master' into master
pjmd89 8efc182
Merge commit '20a52d0a709c9df0a12d99d51814b5beab3cb561' into fixbug/v…
pjmd89 8d053e9
add string
pjmd89 e1babba
add changes
pjmd89 d8ccb8b
change module name
pjmd89 1b28ab3
change all module name
pjmd89 441e566
Add helper functions and short circuit to avoid performance regression
StevenACoffman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure why you removed the
kind == reflect.String
condition as "45.230343" strings are often how they are serialized.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the tests return an error because you are giving a string to an Int.
I removed the string from the int and float because if you set a variable as int or float you should get an int or a float, not a string, in fact, I got errors for that too. I hope it is useful for something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still, adding the string without removing the validation on the Int, there is no problem. the bad thing is that it can return a string in an int or float.