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.

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

Added human-readable message to assert #1531

Merged
merged 2 commits into from Jan 14, 2019

Conversation

Projects
2 participants
@jmjatlanta
Copy link
Contributor

commented Jan 14, 2019

Fixes #1018

This makes the FC_ASSERT related to whitelisting / blacklisting intelligible to the end user.

Note: There are plenty more of these to do. See #691

@abitmore
Copy link
Member

left a comment

Thanks. Best if can wrap the long lines.

@abitmore abitmore added this to the 201901 - Feature Release milestone Jan 14, 2019

@abitmore abitmore added this to In progress in Feature Release (201902) via automation Jan 14, 2019

Feature Release (201902) automation moved this from In progress to Reviewer approved Jan 14, 2019

@jmjatlanta jmjatlanta merged commit 596d7e9 into develop Jan 14, 2019

0 of 3 checks passed

ci/dockercloud Your tests are pending in Docker Cloud
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

Feature Release (201902) automation moved this from Reviewer approved to Done Jan 14, 2019

@jmjatlanta jmjatlanta deleted the jmj_1018 branch Jan 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.