style(clang-tidy): fix modernize-use-integer-sign-comparison checks (#1811)

Signed-off-by: Balakrishna Avulapati <ba@bavulapati.com>
Co-authored-by: Juan Cruz Viotti <jv@jviotti.com>

Author: bavulapati

src/extension/alterschema/alterschema.cc

@@ -6,6 +6,7 @@
 #include <algorithm>
 #include <cmath>
 #include <iterator>
+#include <utility>
 namespace sourcemeta::core {
 static auto
 contains_any(const Vocabularies &container,

src/extension/alterschema/canonicalizer/min_properties_covered_by_required.h

@@ -26,9 +26,9 @@ class MinPropertiesCoveredByRequired final : public SchemaTransformRule {
            schema.at("minProperties").is_integer() &&
            schema.defines("required") && schema.at("required").is_array() &&
            schema.at("required").unique() &&
-           schema.at("required").size() >
-               static_cast<std::uint64_t>(
-                   schema.at("minProperties").to_integer());
+           std::cmp_greater(schema.at("required").size(),
+                            static_cast<std::uint64_t>(
+                                schema.at("minProperties").to_integer()));
   }
 
   auto transform(JSON &schema) const -> void override {

src/extension/alterschema/linter/unsatisfiable_min_properties.h

@@ -26,9 +26,9 @@ class UnsatisfiableMinProperties final : public SchemaTransformRule {
            schema.at("minProperties").is_integer() &&
            schema.defines("required") && schema.at("required").is_array() &&
            schema.at("required").unique() &&
-           schema.at("required").size() >=
-               static_cast<std::uint64_t>(
-                   schema.at("minProperties").to_integer());
+           std::cmp_greater_equal(schema.at("required").size(),
+                                  static_cast<std::uint64_t>(
+                                      schema.at("minProperties").to_integer()));
   }
 
   auto transform(JSON &schema) const -> void override {