ENH: Add new "default" engine argument when notes is not None (openpyxl engine) (pandas-dev#58070)

Co-Authored-By: Dacops <[email protected]>

Author: diogomsmiranda

pandas/io/excel/_base.py

@@ -510,17 +510,14 @@ def read_excel(
     if engine_kwargs is None:
         engine_kwargs = {}
 
-        # set to false so cells have a comment attribute
-        if notes is not None and engine == "openpyxl":
-            engine_kwargs = {"read_only": False}
-
     if not isinstance(io, ExcelFile):
         should_close = True
         io = ExcelFile(
             io,
             storage_options=storage_options,
             engine=engine,
             engine_kwargs=engine_kwargs,
+            notes=notes,
         )
     elif engine and engine != io.engine:
         raise ValueError(
@@ -1606,6 +1603,7 @@ def __init__(
         engine: str | None = None,
         storage_options: StorageOptions | None = None,
         engine_kwargs: dict | None = None,
+        notes: DataFrame | None = None,
     ) -> None:
         if engine_kwargs is None:
             engine_kwargs = {}
@@ -1651,6 +1649,10 @@ def __init__(
         assert engine is not None
         self.engine = engine
         self.storage_options = storage_options
+        
+        # set to false so cells have a comment attribute
+        if notes is not None and engine=='openpyxl':
+            engine_kwargs = {**{"read_only": False},**engine_kwargs}
 
         self._reader = self._engines[engine](
             self._io,