-
Notifications
You must be signed in to change notification settings - Fork 96
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.
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
Rebased "fall back on /etc/riak-cs/app.config when no generated one is found" #1263
Conversation
case re:run(X, Pattern) of | ||
{match, _} -> false; | ||
nomatch -> true | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?assertEqual(nomatch, re:run(..))
may be a little more concise? (and remove the first ?assert
and replace lists:all()
with list comprehension or lists:foreach
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, revert my comment above. The current macro exhibit better correspondence with assertMatchAny
. Sorry for noice.
Updated for the review comments, and fixed small other bugs. |
Rebased "fall back on /etc/riak-cs/app.config when no generated one is found" Reviewed-by: shino
@borshop merge |
This is a recreated PR to 2.1 branch, and adds a simple riak-test for riak-cs-debug.
original PR is #1261