henry-sarabia / sliceconv Goto Github PK

Adopting semantic versioning using git tags will allow users to differentiate versions of the package and, more importantly, identify when the package has made a breaking change.

The table driven tests for the Atoi function are missing the error cases; cases where one or more of the provided strings are not integers. These cases should be added for full test coverage.

As of now, the README only describes the usage of the package between integers and strings. It should be updated to reflect the newly added support for floats and strings as well.

The current package only supports slice conversion between strings and integers. Add support for float64 as well.

When issue #18 is resolved, the README needs to be updated to reflect the changes. One of the examples lists the current accepted boolean string representations.

The current package only supports slice conversion between strings and integers. Add support for bool as well.

Although variadic parameters seemed like a good idea at the time of creating the package, they actually lead to extra boilerplate when calling the package functions. The sliceconv package is meant for converting slices of primitive types to another slice of a primitive type; the use-case of converting a single variable is already covered under the regular strconv package and is out of scope for this package.

Package functions should simply take a slice of primitive types.

There is no README for this package yet. Add a README file with information on what the package is used for, how to use it and how to contribute. Also add build status, test coverage, and Go report card score.

As of v1.0.0, only "true", "True", and "TRUE" are valid string representations of boolean value true. This is the same for the false boolean value. There is no reason to not accept any combination of upper and lowercase characters.

Starting on version 1.0.2, sliceconv will be its own module.

As of now, the README only describes the usage of the package between integers, floats, and strings. It should be updated to reflect the newly added support for bools and strings as well.

This package is not integrated with Travis CI and Coveralls yet so its build and test coverage are unknown to other GitHub users. Add a .travis.yml file and activate coverage.