Enforce ruff/Pylint Warnings (PLW) (#10458)

Author: DimitriPapadopoulos

pyproject.toml

@@ -261,6 +261,7 @@ extend-select = [
   "W",    # pycodestyle warnings
   "PGH",  # pygrep-hooks
   "PLE",  # Pylint Errors
+  "PLW",  # Pylint Warnings
   "UP",   # pyupgrade
   "FURB", # refurb
   "RUF",
@@ -276,6 +277,10 @@ ignore = [
   "PERF203", # try-except within a loop incurs performance overhead
   "E402",    # module level import not at top of file
   "E731",    # do not assign a lambda expression, use a def
+  "PLW0603", # using the global statement to update is discouraged
+  "PLW0642", # reassigned `self` variable in instance method
+  "PLW1641", # object does not implement `__hash__` method
+  "PLW2901", # `for` loop variable overwritten by assignment target
   "UP007",   # use X | Y for type annotations
   "FURB105", # unnecessary empty string passed to `print`
   "RUF001",  # string contains ambiguous unicode character

xarray/backends/chunks.py

@@ -51,7 +51,7 @@ def align_nd_chunks(
 
         # The ideal size of the chunks is the maximum of the two; this would avoid
         # that we use more memory than expected
-        max_chunk = max(fixed_chunk, max(var_chunks))
+        max_chunk = max(fixed_chunk, *var_chunks)
 
         # The algorithm assumes that the chunks on this array are aligned except the last one
         # because it can be considered a partial one