-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix doctests #122
Fix doctests #122
Conversation
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.
Great idea! Did you see that Python 3.6 and 3.7 are causing a seg fault?
Not yet. But that's why I tried it on a branch first 😊 |
574113f
to
9a2da62
Compare
b084a6e
to
952bedc
Compare
70a062b
to
ae1d459
Compare
@amolenaar I made some updates to match the tox/build config from the generic repo, and I fixed an issue setting the strength of a Variable instance. We are getting a seg fault still on the CI, any ideas? |
Signed-off-by: Dan Yeaw <dan@yeaw.me>
Signed-off-by: Dan Yeaw <dan@yeaw.me>
Signed-off-by: Dan Yeaw <dan@yeaw.me>
Signed-off-by: Dan Yeaw <dan@yeaw.me>
Signed-off-by: Dan Yeaw <dan@yeaw.me>
Signed-off-by: Dan Yeaw <dan@yeaw.me>
3223625
to
1e6d5fd
Compare
Signed-off-by: Dan Yeaw <dan@yeaw.me>
3bb34d5
to
734d029
Compare
Sourcery Code Quality Report✅ Merging this PR will increase code quality in the affected files by 0.08%.
Here are some functions in these files that still need a tune-up:
Legend and ExplanationThe emojis denote the absolute quality of the code:
The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request. Please see our documentation here for details on how these metrics are calculated. We are actively working on this report - lots more documentation and extra metrics to come! Let us know what you think of it by mentioning @sourcery-ai in a comment. |
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Doctests were not executed.
Issue Number: N/A
What is the new behavior?
Doctests are executed. Although I think a bunch of them should move to unit tests, since they have little value for (new) users.
Does this PR introduce a breaking change?
Other information