Merge pull request pytest-dev#4552 from nicoddemus/review-changelog-entries

Review changelog entries for features branch

Author: nicoddemus

changelog/3050.deprecation.rst

@@ -1 +1,3 @@
-Deprecate ``pytest.config`` global.  See https://docs.pytest.org/en/latest/deprecations.html#pytest-config-global
+Deprecated the ``pytest.config`` global.
+
+See https://docs.pytest.org/en/latest/deprecations.html#pytest-config-global for rationale.

changelog/3079.removal.rst

@@ -1,3 +1,3 @@
-Remove support for yield tests - they are fundamentally broken because they don't support fixtures properly since collection and test execution were separated.
+Removed support for yield tests - they are fundamentally broken because they don't support fixtures properly since collection and test execution were separated.
 
 See our `docs <https://docs.pytest.org/en/latest/deprecations.html#yield-tests>`__ on information on how to update your code.

changelog/3083.removal.rst

@@ -1,3 +1,3 @@
-Remove ``Metafunc.addcall``. This was the predecessor mechanism to ``@pytest.mark.parametrize``.
+Removed ``Metafunc.addcall``. This was the predecessor mechanism to ``@pytest.mark.parametrize``.
 
 See our `docs <https://docs.pytest.org/en/latest/deprecations.html#metafunc-addcall>`__ on information on how to update your code.

changelog/3191.feature.rst

@@ -1,16 +1,22 @@
 A warning is now issued when assertions are made for ``None``.
 
-This is a common source of confusion among new users, which write::
+This is a common source of confusion among new users, which write:
 
-    assert mocked_object.assert_called_with(3, 4, 5, key='value')
+.. code-block:: python
 
-When they should write::
+    assert mocked_object.assert_called_with(3, 4, 5, key="value")
 
-    mocked_object.assert_called_with(3, 4, 5, key='value')
+When they should write:
+
+.. code-block:: python
+
+    mocked_object.assert_called_with(3, 4, 5, key="value")
 
 Because the ``assert_called_with`` method of mock objects already executes an assertion.
 
-This warning will not be issued when ``None`` is explicitly checked. An assertion like::
+This warning will not be issued when ``None`` is explicitly checked. An assertion like:
+
+.. code-block:: python
 
     assert variable is None
 

changelog/3616.removal.rst

@@ -1,3 +1,3 @@
-Remove the deprecated compat properties for ``node.Class/Function/Module`` - use ``pytest.Class/Function/Module`` now.
+Removed the deprecated compat properties for ``node.Class/Function/Module`` - use ``pytest.Class/Function/Module`` now.
 
 See our `docs <https://docs.pytest.org/en/latest/deprecations.html#internal-classes-accessed-through-node>`__ on information on how to update your code.

changelog/4278.feature.rst

@@ -0,0 +1,4 @@
+``CACHEDIR.TAG`` files are now created inside cache directories.
+
+Those files are part of the `Cache Directory Tagging Standard <http://www.bford.info/cachedir/spec.html>`__, and can
+be used by backup or synchronization programs to identify pytest's cache directory as such.

changelog/4278.trivial.rst

@@ -1 +1 @@
-``pytest.outcomes.Exit`` is derived from ``SystemExit`` instead of ``KeyboardInterrupt``.
+``pytest.outcomes.Exit`` is derived from ``SystemExit`` instead of ``KeyboardInterrupt``. This allows us to better handle ``pdb`` exiting.

changelog/4292.feature.rst

@@ -1 +1 @@
-Restructure ExceptionInfo object construction and ensure incomplete instances have a ``repr``/``str``.
+Restructured ``ExceptionInfo`` object construction and ensure incomplete instances have a ``repr``/``str``.

changelog/4386.feature.rst

@@ -1,4 +1,4 @@
-pdb: support keyword arguments with ``pdb.set_trace``
+pdb: added support for keyword arguments with ``pdb.set_trace``.
 
 It handles ``header`` similar to Python 3.7 does it, and forwards any
 other keyword arguments to the ``Pdb`` constructor.