Use checkboxes as bullet points for the PR review checklists

timhoffm · timhoffm · commit 542e4cab4ad2 · 2019-09-30T01:50:56.000+02:00
diff --git a/doc/devel/coding_guide.rst b/doc/devel/coding_guide.rst
@@ -1,5 +1,13 @@
 .. _pr-guidelines:
 
+.. raw:: html
+
+   <style>
+   .checklist { list-style: none; padding: 0; margin: 0; }
+   .checklist li { padding-left: 24px; margin-right: 6px; }
+   .checklist li:before { content: "\2751"; }
+   </style>
+
 ***********************
 Pull request guidelines
 ***********************
@@ -23,6 +31,8 @@ Summary for PR authors
 
 When making a PR, pay attention to:
 
+.. rst-class:: checklist
+
 * :ref:`Target the master branch <pr-branch-selection>`.
 * Adhere to the :ref:`coding_guidelines`.
 * Update the :ref:`documentation <pr-documentation>` if necessary.
@@ -47,13 +57,17 @@ Summary for PR reviewers
 
 Content topics:
 
+.. rst-class:: checklist
+
 * Is the feature / bugfix reasonable?
 * Does the PR conform with the :ref:`coding_guidelines`?
 * Is the :ref:`documentation <pr-documentation>` (docstrings, examples,
   what's new, API changes) updated?
 
 Organizational topics:
 
+.. rst-class:: checklist
+
 * Make sure all :ref:`automated tests <pr-automated-tests>` pass.
 * The PR should :ref:`target the master branch <pr-branch-selection>`.
 * Tag with descriptive :ref:`labels <pr-labels>`.
