Flexible separators, informative errors

alanhoyle commented 4 years ago

This does a few things:

Excludes .eggs/ from the git repo to help developers
Adds regular-expression-based splitting to allow any non-digit separator (possibly except backslashes?)
If the entry has nulls at the beginning or end, it discards those nulls.
If the entry is too long, it truncates to the first N inputs (and gives a warning)
If the entry is too short, it says so and prompts for reentry
If the entry is the right length, but invalid, it gives a warning to retry with the range of accepted values
Improves formatting of warnings and fixes a grammar issue.

This should address issue #74.

I have modified the original tests so that the new errors pass.

I did not introduce new tests. I suggest that the following strings could be test cases:

Accept:

1 2 3 4 5
1,2,3,4,5
,1,2,3,4,5,

Accept with warning:

1,2,3,4,5,6,

Reject (too short):

,1,2,
1,2,3,4,
11,2

Reject (value out of range):

11,2,3,4,5
1,22,3,4,5

alanhoyle commented 4 years ago

I really appreciate your patience with me on this. Thanks @ulif!

alanhoyle commented 4 years ago

I will start over from scratch and make a simpler version.

ulif / diceware