auto-parse doesn't work. autoParse('123 Words') === NaN. autoParse('123 Words'.substr(0, 3)) would work, but I don't think autoParse is necessary at all. It adds unneeded complexity to a simple task, and if you step through the code, it boils down to new Number(string) anyways. No need for the extra library.
I did not update the version. I believe this can easily be rolled into your next release.
Description
Replaces usage of autoParse with parseInt, and focuses on the first three characters of the line. The use of the magic number 3 is in keeping with line[3] === ' ' from a previous line. 🦎 Chameleon coding.
Motivation and Context
I was not able to send myself an email using simple examples.
How Has This Been Tested?
I modified the code, in situ, and I was able to send myself an email.
Screenshots (if appropriate):
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
auto-parse doesn't work.
autoParse('123 Words') === NaN
.autoParse('123 Words'.substr(0, 3))
would work, but I don't think autoParse is necessary at all. It adds unneeded complexity to a simple task, and if you step through the code, it boils down tonew Number(string)
anyways. No need for the extra library.I did not update the version. I believe this can easily be rolled into your next release.
Description
Replaces usage of
autoParse
withparseInt
, and focuses on the first three characters of the line. The use of the magic number 3 is in keeping withline[3] === ' '
from a previous line. 🦎 Chameleon coding.Motivation and Context
I was not able to send myself an email using simple examples.
How Has This Been Tested?
I modified the code, in situ, and I was able to send myself an email.
Screenshots (if appropriate):
Types of changes
Checklist: