-
Notifications
You must be signed in to change notification settings - Fork 229
Unpin docformatter 1.5.1 #2538
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
Unpin docformatter 1.5.1 #2538
Conversation
Docformatter=1.7.0 has a `--black` option to make it compatible with the black linter. Also updated a few docstrings in the unit test where docformatter tried to capitalize the first letter.
Yeah, let's keep diff --git a/examples/gallery/histograms/scatter_and_histograms.py b/examples/gallery/histograms/scatter_and_histograms.py
index 2c414f7e7..872b4e0d1 100644
--- a/examples/gallery/histograms/scatter_and_histograms.py
+++ b/examples/gallery/histograms/scatter_and_histograms.py
@@ -1,11 +1,11 @@
"""
-Scatter plot with histograms
-----------------------------
-To create a scatter plot with histograms at the sides of the plot one
-can use :meth:`pygmt.Figure.plot` in combination with
-:meth:`pygmt.Figure.histogram`. The positions of the histograms are plotted
-by offseting them from the main scatter plot figure using
-:meth:`pygmt.Figure.shift_origin`.
+Scatter plot with histograms ---------------------------- To create a scatter
+plot with histograms at the sides of the plot one can use
+:meth:`pygmt.Figure.plot` in combination with :meth:`pygmt.Figure.histogram`.
+
+The positions of the histograms are plotted by offseting them from the main
+scatter plot figure using
+:meth: `pygmt.Figure.shift_origin`.
"""
import numpy as np
diff --git a/examples/projections/azim/azim_equidistant.py b/examples/projections/azim/azim_equidistant.py
index 2b75b9266..bee76ffcd 100644 Looking at https://github.com/PyCQA/docformatter/issues, it looks like they're sorting out some issues with whitespace and newlines, so give them a few days. |
FYI, there is a
shouldn't capitalize those words in your docstrings. Or set this list in your pyproject.toml:
|
Shorten some docstrings to remove whitespace.
Style checks passing now with |
Skip docformatter=1.7.1 as it was a bit buggy, see docformatter=1.7.2 release notes at https://github.com/PyCQA/docformatter/releases/tag/v1.7.2.
Description of proposed changes
Docformatter=1.7.0 has a
--black
option to make it compatible with the black linter. Also updated a few docstrings in the unit test where docformatter tried to capitalize the first letter.Thanks @seisman for checking the docformatter 1.7.0 RC releases too at PyCQA/docformatter#199!
Reverts #2482, resolves #2476.
Reminders
make format
andmake check
to make sure the code follows the style guide.doc/api/index.rst
.Slash Commands
You can write slash commands (
/command
) in the first line of a comment to performspecific operations. Supported slash commands are:
/format
: automatically format and lint the code/test-gmt-dev
: run full tests on the latest GMT development version