Reportlab made optional & updated log parsing

Author: dadyownes15

carbontracker/cli.py

@@ -16,7 +16,7 @@ def generate_report(log_file, output_pdf):
     # Check for reportlab dependency early
     if not REPORTLAB_AVAILABLE:
         print("Error: The 'reportlab' package is required to generate PDF reports but is not installed.")
-        print("Please install it with: pip install carbontracker[pdfreport]")
+        print("Please install it with: pip install 'carbontracker[pdfreport]'")
         return
 
     if not os.path.exists(log_file):

carbontracker/report.py

@@ -226,7 +226,7 @@ def generate_report_from_log(log_file_path, output_path):
     if not REPORTLAB_AVAILABLE:
         raise ImportError(
             "The 'reportlab' package is required to generate PDF reports but is not installed. "
-            "Please install it with: pip install carbontracker[pdfreport]"
+            "Please install it with: pip install 'carbontracker[pdfreport]'"
         )
 
     # Read and parse log

tests/test_cli.py

@@ -87,7 +87,7 @@ def test_generate_report_shows_error_without_reportlab(self):
             output = captured_output.getvalue()
 
             # Check the error message contains installation instructions
-            self.assertIn("pip install carbontracker[pdfreport]", output)
+            self.assertIn("pip install 'carbontracker[pdfreport]'", output)
             self.assertIn("reportlab", output)
         finally:
             # Restore original values

tests/test_report.py

@@ -279,7 +279,7 @@ def test_generate_report_raises_import_error_without_reportlab(self):
                 generate_report_from_log("dummy_log.txt", "dummy_output.pdf")
 
             # Check the error message contains installation instructions
-            self.assertIn("pip install carbontracker[pdfreport]", str(context.exception))
+            self.assertIn("pip install 'carbontracker[pdfreport]'", str(context.exception))
             self.assertIn("reportlab", str(context.exception))
         finally:
             # Restore original value