pytest.skip() - remove kwarg entirely, to avoid breaking older pytest and not getting warns from newer pytest

tests/test_metadata.py

@@ -188,7 +188,7 @@ def test_xml_metadata_preserved(test_file, output_type, resources, outpdf):
     try:
         from libxmp.utils import file_to_dict  # pylint: disable=import-outside-toplevel
     except Exception:  # pylint: disable=broad-except
-        pytest.skip(reason="libxmp not available or libexempi3 not installed")
+        pytest.skip("libxmp not available or libexempi3 not installed")
 
     before = file_to_dict(str(input_file))
 

tests/test_stdio.py

@@ -35,7 +35,7 @@ def test_stdin(ocrmypdf_exec, resources, outpdf):
 
 def test_stdout(ocrmypdf_exec, resources, outpdf):
     if 'COV_CORE_DATAFILE' in os.environ:
-        pytest.skip(reason="Coverage uses stdout")
+        pytest.skip("Coverage uses stdout")
 
     input_file = str(resources / 'francais.pdf')
     output_file = str(outpdf)
@@ -72,7 +72,7 @@ def test_bad_locale(monkeypatch):
 )
 def test_dev_null(resources):
     if 'COV_CORE_DATAFILE' in os.environ:
-        pytest.skip(reason="Coverage uses stdout")
+        pytest.skip("Coverage uses stdout")
 
     p = run_ocrmypdf(
         resources / 'trivial.pdf',