lint + format

Author: james-d-mitchell

src/order.cpp

@@ -1409,8 +1409,8 @@ Reinitialize a {0} object from a {1} list.
 )pbdoc",
                                                                   name);
       py::class_<Cmp>   thing(m,
-                              binding_name,
-                              fmt::format(R"pbdoc(
+                            binding_name,
+                            fmt::format(R"pbdoc(
 Compare words using {2}.
 
 Use ``{0}({1})`` to compare either ``str`` or ``list[int]`` words. In this
@@ -1438,11 +1438,11 @@ type as *alphabet*.
   True
 {3}
 )pbdoc",
-                                          name,
-                                          configuration,
-                                          ordering,
-                                          non_alphabet_string_doctest)
-                                  .c_str());
+                                        name,
+                                        configuration,
+                                        ordering,
+                                        non_alphabet_string_doctest)
+                                .c_str());
 
       thing.def(py::init<Alphabet<Word> const&, std::vector<size_t> const&>(),
                 py::arg("alphabet"),
@@ -2836,7 +2836,7 @@ Return the alphabet used to compare letters.
 )pbdoc",
                 py::return_value_policy::reference_internal);
     }  // bind_rev_lenlex_cmp_with_alphabet
-  }  // namespace
+  }    // namespace
 
   void init_order(py::module& m) {
     py::options options;

tests/test_order.py

@@ -4,6 +4,8 @@
 #
 # The full license is in the file LICENSE, distributed with this software.
 
+# pylint: disable=too-many-lines, invalid-name
+
 """Tests for order.cpp."""
 
 from copy import copy
@@ -60,7 +62,7 @@ def _check_simple_copy_and_repr(Cmp, name):
             original.__copy__(),  # pylint: disable=unnecessary-dunder-call
             copy(original),
         )
-        assert all(type(copied) is Cmp for copied in copies)
+        assert all(isinstance(copied, Cmp) for copied in copies)
         assert all(copied is not original for copied in copies)
         assert all(copied.py_template_params == original.py_template_params for copied in copies)
         assert all(copied(x, y) for copied in copies)
@@ -90,7 +92,7 @@ def _check_configured_copy_and_repr(Cmp, name, configuration_name, accessor_name
             copy(original),
         )
         original.init(*replacement_args)
-        assert all(type(copied) is Cmp for copied in copies)
+        assert all(isinstance(copied, Cmp) for copied in copies)
         assert all(copied is not original for copied in copies)
         assert all(getattr(copied, accessor_name)() == [1, 2] for copied in copies)
         if alphabet is not None: