remove redundant comments

Author: juleswg23

nbmail/compose/blocks.py

@@ -202,7 +202,6 @@ def block_text(
         column(
             mjml_text(content=html, attributes={"align": align}),
         ),
-        # attributes={"padding": "0px"},
     )
 
     return Block(mjml_section)
@@ -248,7 +247,6 @@ def block_title(
                 attributes={"align": align},
             )
         ),
-        # attributes={"padding": "0px"},
     )
 
     return Block(mjml_section)
@@ -285,7 +283,6 @@ def block_spacer(height: str = "20px") -> Block:
         column(
             mjml_spacer(attributes={"height": height}),
         ),
-        # attributes={"padding": "0px"},
     )
 
     return Block(mjml_section)
@@ -354,32 +351,26 @@ def block_image(
     - Images from local files are converted to inline attachments with CID references
       during email processing by mjml_to_email().
     - If `float` is not "none", it takes precedence and overrides `align`.
-    """    
-    # Convert integer width to CSS string
+    """
     if isinstance(width, int):
         width_str = f"{width}px"
     else:
         width_str = width
 
-    # Determine alignment style based on float and align parameters
     align_style = ""
 
-    # Float takes precedence if not "none"
     if float != "none":
         align_style = f"float: {float};"
     else:
-        # Use align parameter if float is "none"
         if align == "center":
             align_style = "display: block; margin: 0 auto;"
         elif align == "left":
             align_style = "display: block; margin: 0;"
         elif align == "right":
             align_style = "display: block; margin: 0 0 0 auto;"
-        # "inline" has no special style
 
     # Detect URL vs local file
     if _is_url(file):
-        # For URLs, use as-is
         src = file
     else:
         # For local files, read as bytes for processing by _process_mjml_images

nbmail/compose/compose.py

@@ -134,7 +134,6 @@ def compose_email(
     )
     ```
     """
-    # Convert sections (header, body, footer) to MJML lists
     header_mjml_list = _component_to_mjml_section(header)
     body_mjml_list = _component_to_mjml_section(body)
     footer_mjml_list = _component_to_mjml_section(footer)
@@ -144,13 +143,11 @@ def compose_email(
         title_block_mjml = block_title(title)._to_mjml()
         header_mjml_list = [title_block_mjml] + header_mjml_list
 
-    # Apply template wrapper if requested
     if template == "blastula":
         all_sections = _apply_blastula_template(
             header_mjml_list, body_mjml_list, footer_mjml_list
         )
     elif template == "none":
-        # Combine all sections without template
         all_sections = header_mjml_list + body_mjml_list + footer_mjml_list
     else:
         raise ValueError(f"Unknown template: {template}. Use 'blastula' or 'none'.")
@@ -174,9 +171,6 @@ def compose_email(
         ),
     )
 
-    # TODO: remove, this is for testing purposes
-    # print(email_structure._to_mjml())
-
     email_obj = mjml_to_email(email_structure)
 
     return email_obj
@@ -211,7 +205,6 @@ def _component_to_mjml_section(
         return component._to_mjml_list()
 
     elif isinstance(component, str):
-        # Convert string to BlockList and then to MJML
         block_list = BlockList(component)
         return block_list._to_mjml_list()
 
@@ -268,7 +261,6 @@ def apply_blastula_styles(sections: list[MJMLTag]) -> list[MJMLTag]:
         for section in body_sections
     ]
 
-    # Wrap body with styling
     body_wrapper = wrapper(
         *styled_body,
         attributes={

nbmail/compose/inline_utils.py

@@ -144,7 +144,6 @@ def _read_local_file_as_data_uri(file: str) -> str:
     with open(file_path, "rb") as f:
         file_bytes = f.read()
 
-    # Encode to base64
     b64_string = base64.b64encode(file_bytes).decode("utf-8")
 
     # Guess MIME type