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

Remove csrng.net dependency #65

Merged

travisbrown merged 3 commits into master from topic/csrng

Apr 28, 2020

Owner

travisbrown commented Apr 28, 2020

This PR does two unrelated things:

Temporarily ignores a couple of tests that depend on csrng.net, which is down. (These tests are being changed to not have this dependency, at which point we'll unignore them.)
Adds a string length check to avoid looking at the first character in a possibly-empty string (right now the argument should never be empty, but this might change).

travisbrown added 3 commits

April 28, 2020 03:40


          Add string length check

a0394d1


          Ignore acceptance tests that depend on csrng.net

bc0d409


          Replace some enum switches

62d642d

travisbrown merged commit 8e24ba8 into master

travisbrown deleted the topic/csrng branch

April 28, 2020 14:29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet