Wrap to 80 characters

Author: gergondet-woven

bazel_ros2_rules/ros2/resources/cmake_tools/file_api.py

@@ -69,11 +69,14 @@ def __init__(self, target_file: Path):
                 if fragment['role'] == 'flags':
                     self.link_flags.append(fragment['fragment'])
                 elif fragment['role'] == 'libraries':
-                    # In the generated JSON a library can be in quotes (to escape some characters?)
-                    # This escaping creates problem down the line ("/usr/lib/libfoo.so" is *not* a path)
+                    # In the generated JSON a library can be in quotes, in
+                    # order to escape some characters.
+                    # This escaping creates problem down the line,
+                    # "/usr/lib/libfoo.so" is *not* a path.
                     # We remove the escaping here so they can be treated normally
                     link_library = fragment['fragment']
-                    if link_library.startswith('"') and link_library.endswith('"'):
+                    if link_library.startswith('"') and \
+                            link_library.endswith('"'):
                         link_library = link_library.strip('"')
                     self.link_libraries.append(link_library)
                 elif fragment['role'] == 'libraryPath':

bazel_ros2_rules/ros2/resources/ros2bzl/scraping/ament_python.py

@@ -14,7 +14,12 @@ def find_python_package(name):
     dist = distribution(name)
     top_level = dist.read_text('top_level.txt')
     top_level = top_level.rstrip('\n')
-    return str(dist._path), '\n'.join([str(dist.locate_file(top_level_i)) for top_level_i in top_level.split('\n')])
+    return str(dist._path), "\n".join(
+            [
+                str(dist.locate_file(top_level_i))
+                for top_level_i in top_level.split("\n")
+            ]
+        )
 
 
 def collect_ament_python_package_properties(name, metadata):

bazel_ros2_rules/ros2/resources/ros2bzl/scraping/metadata.py

@@ -9,9 +9,9 @@ def remove_ros1_elements(root):
     for parent in root.iter():
         for child in list(parent):
             if "condition" in child.attrib:
-                if child.get('condition') == ros1_condition_value :
+                if child.get('condition') == ros1_condition_value:
                     elements_to_remove.append((parent, child))
-                else :
+                else:
                     child.attrib.pop("condition")
 
     for parent, child in elements_to_remove:

bazel_ros2_rules/ros2/resources/ros2bzl/templates.py

@@ -151,7 +151,10 @@ def configure_package_py_library(
     target_name = py_name(name, metadata)
     eggs = [sandbox(egg_path) for egg_path, _ in properties.python_packages]
     tops = [
-        sandbox(top_level_i) for _, top_level in properties.python_packages for top_level_i in top_level.split("\n")]
+        sandbox(top_level_i)
+        for _, top_level in properties.python_packages
+        for top_level_i in top_level.split("\n")
+    ]
     imports = [os.path.dirname(egg) for egg in eggs]
 
     template = 'templates/package_py_library.bazel.tpl'

bazel_ros2_rules/ros2/resources/templates/package_cc_library.bazel.tpl

@@ -1,7 +1,11 @@
 cc_library(
     name = @name@,
     srcs = @srcs@,
-    hdrs = glob(["{}/**/*.h*".format(x) for x in @headers@] + ["{}/**/*.inc".format(x) for x in @headers@], allow_empty = True),
+    hdrs = glob(
+        ["{}/**/*.h*".format(x) for x in @headers@] +
+        ["{}/**/*.inc".format(x) for x in @headers@],
+        allow_empty = True,
+    ),
     includes = @includes@,
     copts = @copts@,
     defines = @defines@,