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
Subclass abcmeta #123
Subclass abcmeta #123
Conversation
Codecov Report
@@ Coverage Diff @@
## master #123 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 9 9
Lines 852 853 +1
Branches 191 191
=====================================
+ Hits 852 853 +1 |
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.
Looking good, we'll need some docs too.
Makefile
Outdated
@@ -19,7 +19,7 @@ lint: | |||
|
|||
.PHONY: test | |||
test: | |||
pytest --cov=pydantic |
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.
remove these changes, the problem should be fixed by Teemu/pytest-sugar#133 which will hopefully be merged soon.
If the problems persist I'll remove pytest-sugar
completely.
What kind of documentation are you thinking about? Since it's in the stdlib the documentation is already there: https://docs.python.org/3/library/abc.html Maybe just a mention in the changelog? |
Just a show section in the main docs saying something like "pydantic works with python's standard abc" then a mini example in python. |
Tests are failing since I readded |
Looks great. Thank you very much. I'll wait a couple of days in the hope that pytest-sugar gets fixed, if it doesn't I'll remove it. Once that's done this looks ready to merge. |
great, thank you very much. |
Expands docstring coverage for #5568 ## Change Summary <!-- Please give a short summary of the changes. --> ## Related issue number <!-- Are there any issues opened that will be resolved by merging this change? --> <!-- WARNING: please use "fix #123" style references so the issue is closed when this PR is merged. --> ## Checklist * [ ] Unit tests for the changes exist * [ ] Tests pass on CI and coverage remains at 100% * [x] Documentation reflects the changes where applicable * [ ] `changes/<pull request or issue id>-<github username>.md` file added describing change (see [changes/README.md](https://github.com/samuelcolvin/pydantic/blob/master/changes/README.md) for details) * [ ] My PR is ready to review, **please add a comment including the phrase "please review" to assign reviewers**
Expands docstring and moves page in API docs to preserve alpha ordering ## Change Summary <!-- Please give a short summary of the changes. --> ## Related issue number <!-- Are there any issues opened that will be resolved by merging this change? --> <!-- WARNING: please use "fix #123" style references so the issue is closed when this PR is merged. --> ## Checklist * [ ] Unit tests for the changes exist * [ ] Tests pass on CI and coverage remains at 100% * [x] Documentation reflects the changes where applicable * [ ] `changes/<pull request or issue id>-<github username>.md` file added describing change (see [changes/README.md](https://github.com/samuelcolvin/pydantic/blob/master/changes/README.md) for details) * [ ] My PR is ready to review, **please add a comment including the phrase "please review" to assign reviewers** Selected Reviewer: @dmontagu
* add complete benchmark * fixing tests * extending benchmarks * tweaks and test readme * skip complete benchmarks for <py3.10
Implemented what we discussed in #120
Also changed pytest execution in the Makefile (was failing in my machine).