chore: upgrade black and pre-commit

Author: tswast

.flake8

@@ -14,9 +14,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Generated by synthtool. DO NOT EDIT!
 [flake8]
-ignore = E203, E231, E266, E501, W503
+ignore = E203, E231, E266, E501, W503, E704
 exclude =
   # Exclude generated code.
   **/proto/**

.github/header-checker-lint.yml

@@ -2,14 +2,14 @@
  "allowedLicenses": ["Apache-2.0", "MIT", "BSD-3"],
  "ignoreFiles": ["**/requirements.txt", "**/requirements-test.txt", "**/__init__.py", "samples/**/constraints.txt", "samples/**/constraints-test.txt"],
  "sourceFileExtensions": [
- 	"ts", 
- 	"js", 
- 	"java", 
- 	"sh", 
- 	"Dockerfile", 
- 	"yaml", 
+ 	"ts",
+ 	"js",
+ 	"java",
+ 	"sh",
+ 	"Dockerfile",
+ 	"yaml",
  	"py",
  	"html",
  	"txt"
  ]
-}
+}

.pre-commit-config.yaml

@@ -16,7 +16,7 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.0.1
+    rev: v5.0.0
     hooks:
     -   id: trailing-whitespace
     -   id: end-of-file-fixer
@@ -28,11 +28,11 @@ repos:
     -   id: isort
         name: isort (python)
 -   repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 25.1.0
     hooks:
     - id: black
 -   repo: https://github.com/pycqa/flake8
-    rev: 7.1.2
+    rev: 7.3.0
     hooks:
     - id: flake8
 -   repo: https://github.com/pre-commit/mirrors-mypy

CODE_OF_CONDUCT.md

@@ -92,4 +92,4 @@ harassment or threats to anyone's safety, we may take action without notice.
 
 This Code of Conduct is adapted from the Contributor Covenant, version 1.4,
 available at
-https://www.contributor-covenant.org/version/1/4/code-of-conduct.html
+https://www.contributor-covenant.org/version/1/4/code-of-conduct.html

OWNERS

@@ -5,4 +5,4 @@ jiaxun@google.com
 mlaurencechen@google.com
 shobs@google.com
 swast@google.com
-tbergeron@google.com
+tbergeron@google.com

bigframes/bigquery/__init__.py

@@ -14,7 +14,7 @@
 
 """This module integrates BigQuery built-in functions for use with DataFrame objects,
 such as array functions:
-https://cloud.google.com/bigquery/docs/reference/standard-sql/array_functions. """
+https://cloud.google.com/bigquery/docs/reference/standard-sql/array_functions."""
 
 from bigframes.bigquery._operations.approx_agg import approx_top_count
 from bigframes.bigquery._operations.array import (

bigframes/bigquery/_operations/struct.py

@@ -15,7 +15,7 @@
 
 """This module integrates BigQuery built-in functions for use with DataFrame objects,
 such as array functions:
-https://cloud.google.com/bigquery/docs/reference/standard-sql/array_functions. """
+https://cloud.google.com/bigquery/docs/reference/standard-sql/array_functions."""
 
 
 from __future__ import annotations

bigframes/core/array_value.py

@@ -524,9 +524,11 @@ def prepare_join_names(
     ) -> Tuple[bigframes.core.nodes.BigFrameNode, dict[str, str]]:
         if set(other.node.ids) & set(self.node.ids):
             r_mapping = {  # Rename conflicting names
-                rcol.name: rcol.name
-                if (rcol.name not in self.column_ids)
-                else bigframes.core.guid.generate_guid()
+                rcol.name: (
+                    rcol.name
+                    if (rcol.name not in self.column_ids)
+                    else bigframes.core.guid.generate_guid()
+                )
                 for rcol in other.node.ids
             }
             return (

bigframes/core/bigframe_node.py

@@ -155,8 +155,7 @@ def roots(self) -> typing.Set[BigFrameNode]:
     # TODO: Store some local data lazily for select, aggregate nodes.
     @property
     @abc.abstractmethod
-    def fields(self) -> Sequence[field.Field]:
-        ...
+    def fields(self) -> Sequence[field.Field]: ...
 
     @property
     def ids(self) -> Iterable[identifiers.ColumnId]:
@@ -322,9 +321,9 @@ def iter_nodes_topo(self: BigFrameNode) -> Generator[BigFrameNode, None, None]:
             [node for node in self.unique_nodes() if not node.child_nodes]
         )
 
-        child_to_parents: Dict[
-            BigFrameNode, Set[BigFrameNode]
-        ] = collections.defaultdict(set)
+        child_to_parents: Dict[BigFrameNode, Set[BigFrameNode]] = (
+            collections.defaultdict(set)
+        )
         for parent, child in self.edges():
             child_to_parents[child].add(parent)
 

bigframes/core/block_transforms.py

@@ -376,9 +376,11 @@ def value_counts(
             [
                 ordering.OrderingExpression(
                     ex.deref(count_id),
-                    direction=ordering.OrderingDirection.ASC
-                    if ascending
-                    else ordering.OrderingDirection.DESC,
+                    direction=(
+                        ordering.OrderingDirection.ASC
+                        if ascending
+                        else ordering.OrderingDirection.DESC
+                    ),
                 )
             ]
         )
@@ -435,22 +437,24 @@ def rank(
         window_ordering = (
             ordering.OrderingExpression(
                 ex.deref(col),
-                ordering.OrderingDirection.ASC
-                if ascending
-                else ordering.OrderingDirection.DESC,
+                (
+                    ordering.OrderingDirection.ASC
+                    if ascending
+                    else ordering.OrderingDirection.DESC
+                ),
                 na_last=(na_option in ["bottom", "keep"]),
             ),
         )
         # Count_op ignores nulls, so if na_option is "top" or "bottom", we instead count the nullity columns, where nulls have been mapped to bools
         block, rownum_id = block.apply_window_op(
             col if na_option == "keep" else nullity_col_id,
             agg_ops.dense_rank_op if method == "dense" else agg_ops.count_op,
-            window_spec=windows.unbound(
-                grouping_keys=grouping_cols, ordering=window_ordering
-            )
-            if method == "dense"
-            else windows.rows(
-                end=0, ordering=window_ordering, grouping_keys=grouping_cols
+            window_spec=(
+                windows.unbound(grouping_keys=grouping_cols, ordering=window_ordering)
+                if method == "dense"
+                else windows.rows(
+                    end=0, ordering=window_ordering, grouping_keys=grouping_cols
+                )
             ),
             skip_reproject_unsafe=(col != columns[-1]),
         )