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.

Sign up for GitHub

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

Jump to bottom

Handle JSON Number in ParseCommaStringSlice #29

Merged
merged 6 commits into from
Mar 16, 2022
Merged

Handle JSON Number in ParseCommaStringSlice #29

merged 6 commits into from
Mar 16, 2022

Commits on Mar 3, 2022

  1. Return err if ParseCommaStringSlice fails to parse input string

    @ccapurso
    ccapurso committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    a641c02 View commit details
    Browse the repository at this point in the history

  2. Add test cases for ParseCommaStringSlice

    @ccapurso
    ccapurso committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    9ff43d8 View commit details
    Browse the repository at this point in the history

Commits on Mar 4, 2022

  1. properly handle json.Number in ParseCommaStringSlice by stringifying

    @ccapurso
    ccapurso committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    509a318 View commit details
    Browse the repository at this point in the history

  2. add exhaustive type support testing for ParseCommaStringSlice

    @ccapurso
    ccapurso committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    cd31ce9 View commit details
    Browse the repository at this point in the history

  3. remove accidental added newline

    @ccapurso
    ccapurso committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    036dca0 View commit details
    Browse the repository at this point in the history

  4. adding a couple more cases to Test_ParseCommaStringSlice

    @ccapurso
    ccapurso committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    94fa554 View commit details
    Browse the repository at this point in the history