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

Use validate_dict for more accurate error messages #4617

Merged
merged 1 commit into from Sep 7, 2018

Conversation

Projects
None yet
3 participants
@invinciblycool
Contributor

invinciblycool commented Sep 7, 2018

Fixes #4530

@stsewd

stsewd approved these changes Sep 7, 2018

Thank you!

@agjohnson

This comment has been minimized.

Show comment
Hide comment
@agjohnson

agjohnson Sep 7, 2018

Contributor

Thanks for the contribution @invinciblycool ! 🎉

Contributor

agjohnson commented Sep 7, 2018

Thanks for the contribution @invinciblycool ! 🎉

@agjohnson agjohnson merged commit 3501ca0 into rtfd:master Sep 7, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@invinciblycool

This comment has been minimized.

Show comment
Hide comment
@invinciblycool

invinciblycool Sep 7, 2018

Contributor

RTD has been a great help for me so thought of giving back to it. Looking forward to contribute more.

Contributor

invinciblycool commented Sep 7, 2018

RTD has been a great help for me so thought of giving back to it. Looking forward to contribute more.

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