-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
Change PR06 logic to only fail when type is used standalone #447
Conversation
As per feedback in numpy#447
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.
I took the liberty of running black to make sure the formatting was consistent but other than that this LGTM. Thanks @amoeba !
The CI failures are unrelated - see pydata/pydata-sphinx-theme#1130. |
I'm going to go ahead and get this in since CI has been patched. Thanks for raising+fixing @amoeba |
Thank you @rossbar! |
* MAINT: Fix CIs for sphinx 6 (numpy#448) * BUG: PR06 logic to only fail when type is used standalone (numpy#447) Change PR06 logic to only fail when type is used standalone. This prevents failures for user-defined classes, e.g. Mystring Co-authored-by: Ross Barnowski <rossbar@berkeley.edu> * Update pre-commit (numpy#449) * Update pre-commit * Run pre-commit on all files * Update blacken-docs repo in pre-commit conf after owner transfer. (numpy#451) * MAINT: Compatibility with pydata-sphinx-theme (numpy#453) Fix compatibility issues between latest pydata-sphinx-theme and sphinx 6. Increase granularity of CI to test across necessary sphinx & Python version combos. * MAINT: Refactoring `get_doc_object` (numpy#452) --------- Co-authored-by: Eric Larson <larson.eric.d@gmail.com> Co-authored-by: Bryce Mecum <petridish@gmail.com> Co-authored-by: Ross Barnowski <rossbar@berkeley.edu> Co-authored-by: Jarrod Millman <jarrod.millman@gmail.com> Co-authored-by: Stefanie Molin <24376333+stefmolin@users.noreply.github.com> Co-authored-by: yuki <126044367+F3eQnxN3RriK@users.noreply.github.com>
* MAINT: Fix CIs for sphinx 6 (numpy#448) * BUG: PR06 logic to only fail when type is used standalone (numpy#447) Change PR06 logic to only fail when type is used standalone. This prevents failures for user-defined classes, e.g. Mystring Co-authored-by: Ross Barnowski <rossbar@berkeley.edu> * Update pre-commit (numpy#449) * Update pre-commit * Run pre-commit on all files * Update blacken-docs repo in pre-commit conf after owner transfer. (numpy#451) * MAINT: Compatibility with pydata-sphinx-theme (numpy#453) Fix compatibility issues between latest pydata-sphinx-theme and sphinx 6. Increase granularity of CI to test across necessary sphinx & Python version combos. * MAINT: Refactoring `get_doc_object` (numpy#452) --------- Co-authored-by: Eric Larson <larson.eric.d@gmail.com> Co-authored-by: Bryce Mecum <petridish@gmail.com> Co-authored-by: Ross Barnowski <rossbar@berkeley.edu> Co-authored-by: Jarrod Millman <jarrod.millman@gmail.com> Co-authored-by: Stefanie Molin <24376333+stefmolin@users.noreply.github.com> Co-authored-by: yuki <126044367+F3eQnxN3RriK@users.noreply.github.com>
Closes #446
This is a draft PR for addressing #446 which hasn't yet been discussed. For reviewing, I'd pay special attention to how I went the logic in
validate.py
, especially the regex.