Skip to content

Commit 57f5908

Browse files
committed
mend
1 parent 6afa9bf commit 57f5908

File tree

3 files changed

+39
-20
lines changed

3 files changed

+39
-20
lines changed

doc/_static/mpl.css

+18-4
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,21 @@ div.wide-table table th.stub {
167167
font-size: large;
168168
}
169169

170-
.checklist { list-style: none; padding: 0; margin: 0; }
171-
.checklist li { margin-left: 24px; padding-left: 23px; margin-right: 6px; }
172-
.checklist li:before { content: "\2610\2001"; margin-left: -24px; }
173-
.checklist li p {display: inline; }
170+
171+
.checklist {
172+
list-style: none;
173+
padding: 0;
174+
margin: 0;
175+
}
176+
.checklist li {
177+
margin-left: 24px;
178+
padding-left: 23px;
179+
margin-right: 6px;
180+
}
181+
.checklist li:before {
182+
content: "\2610\2001";
183+
margin-left: -24px;
184+
}
185+
.checklist li p {
186+
display: inline;
187+
}

doc/devel/coding_guide.rst

+3-2
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ on GitHub and specify what feedback from the developers would be helpful.
1717

1818
Please be patient with reviewers. We try our best to respond quickly, but we have
1919
limited bandwidth. If there is no feedback within a couple of days, please ping
20-
us by posting a comment to your PR.
20+
us by posting a comment to your PR or reaching out on a :ref:`communication channel <communication-channels>`
2121

2222

2323
Summary for pull request authors
@@ -213,7 +213,8 @@ Merging
213213

214214
Automated tests
215215
---------------
216-
Before being merged, a PR should pass the :ref:`automated-tests`. If you are unsure why a test is failing, ask on the PR or in our `chat space <https://gitter.im/matplotlib/matplotlib>`_
216+
Before being merged, a PR should pass the :ref:`automated-tests`. If you are
217+
unsure why a test is failing, ask on the PR or in our :ref:`communication-channels`
217218

218219
.. _pr-squashing:
219220

doc/devel/contribute.rst

+18-14
Original file line numberDiff line numberDiff line change
@@ -416,18 +416,22 @@ Variable names
416416
When feasible, please use our internal variable naming convention for objects
417417
of a given class and objects of any child class:
418418

419-
+---------------+------------------------------------+
420-
| variable name | base class |
421-
+===============+====================================+
422-
| ``fig`` | `~matplotlib.figure.FigureBase` |
423-
+---------------+------------------------------------+
424-
| ``ax`` | `~matplotlib.axes.Axes` |
425-
+---------------+------------------------------------+
426-
| ``trans`` | `~matplotlib.transforms.Transform` |
427-
+---------------+------------------------------------+
428-
429-
Generally, denote more than instance of the same class by adding numbers to
430-
the variable names, i.e. ``trans1``, ``trans2``.
419+
+------------------------------------+---------------+------------------------------------------+
420+
| base class | variable | multiples |
421+
+====================================+===============+==========================================+
422+
| `~matplotlib.figure.FigureBase` | ``fig`` | |
423+
+------------------------------------+---------------+------------------------------------------+
424+
| `~matplotlib.axes.Axes` | ``ax`` | |
425+
+------------------------------------+---------------+------------------------------------------+
426+
| `~matplotlib.transforms.Transform` | ``trans`` | ``trans_<source>_<target>`` |
427+
+ + + +
428+
| | | ``trans_<source>`` when target is screen |
429+
+------------------------------------+---------------+------------------------------------------+
430+
431+
Generally, denote more than one instance of the same class by adding suffixes to
432+
the variable names. If a format isn't specified in the table, use numbers or
433+
letters as appropriate.
434+
431435

432436
.. _type-hints:
433437

@@ -495,13 +499,13 @@ ensure that a reference is included in the descriptive text.
495499

496500
API Change Notes
497501
""""""""""""""""
498-
.. include:: doc/api/next_api_changes/readme.rst
502+
.. include:: ../api/next_api_changes/README.rst
499503
:start-line: 5
500504
:end-line: 31
501505

502506
What's new
503507
""""""""""
504-
.. include:: doc/users/next_whats_new/readme.rst
508+
.. include:: ../users/next_whats_new/README.rst
505509
:start-line: 5
506510
:end-line: 24
507511

0 commit comments

Comments
 (0)