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

More helpful error messages. #47

Closed
wants to merge 4 commits into from
Closed

Conversation

Cediddi
Copy link

@Cediddi Cediddi commented May 26, 2017

This is an example solution to #46 . I also added a test case too.

@coveralls
Copy link

coveralls commented May 26, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling e7b6fbb on genomize:master into 56e5a27 on clarkduvall:master.

@Cediddi
Copy link
Author

Cediddi commented Feb 21, 2018

This pr is very small and helpful in exceptional situations. Would you kindly review and merge it, maybe?

@lisael
Copy link

lisael commented Nov 12, 2019

There's only a flake8 error on a test method redefinition. @Cediddi would you mind renaming test_optional_methodfield (or remove the original one if the two tests overlap), so this could be merged ?

@Cediddi
Copy link
Author

Cediddi commented Nov 25, 2019

I'll do the changes as soon as possible.

@Cediddi
Copy link
Author

Cediddi commented Jan 11, 2020

I've opened a new PR for #46 and #73 together. Closing this now.

@Cediddi Cediddi closed this Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants