-
Notifications
You must be signed in to change notification settings - Fork 31
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
Fixing blacken-docs error messages #551
base: main
Are you sure you want to change the base?
Fixing blacken-docs error messages #551
Conversation
@@ -781,11 +781,14 @@ def transform2D(self: SameVectorType, obj: TransformProtocol2D) -> SameVectorTyp | |||
""" | |||
Arbitrarily transforms the vector(s) by | |||
|
|||
.. blacken-docs:off |
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.
To maintain consistency in the _methods.py
file, I have used .. blacken-docs:off/on
here instead of replacing python
with pycon
. LMKWYT.
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.
Thanks for looking into this, @Schefflera-Arboricola!
I would be happy with this, or with @jpivarski's solution of closing the issue without any modifications (as now this is the "dev" docstrings).
I would let @jpivarski decide whichever is better!
Description
Kindly take a look at CONTRIBUTING.md.
Please describe the purpose of this pull request. Reference and link to any relevant issues or pull requests.
closes #240
Note for reviewer
Reviewing this PR commit-wise might be easier.
Checklist
$ pre-commit run --all-files
or$ nox -s lint
)$ pytest
or$ nox -s tests
)$ cd docs; make clean; make html
or$ nox -s docs
)$ pytest --doctest-plus src/vector/
or$ nox -s doctests
)Before Merging