Make all *Pool constructors take only keyword arguments

There's going to be more parameters soon, so this would be make it
easier to read.

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

src/frequenz/sdk/microgrid/_data_pipeline.py

@@ -261,7 +261,9 @@ def ev_charger_pool(
                 )
             )
         return EVChargerPool(
-            self._ev_charger_pool_reference_stores[ref_store_key], name, priority
+            pool_ref_store=self._ev_charger_pool_reference_stores[ref_store_key],
+            name=name,
+            priority=priority,
         )
 
     def pv_pool(
@@ -328,7 +330,11 @@ def pv_pool(
                 component_ids=component_ids,
             )
 
-        return PVPool(self._pv_pool_reference_stores[ref_store_key], name, priority)
+        return PVPool(
+            pool_ref_store=self._pv_pool_reference_stores[ref_store_key],
+            name=name,
+            priority=priority,
+        )
 
     def battery_pool(
         self,
@@ -400,7 +406,9 @@ def battery_pool(
             )
 
         return BatteryPool(
-            self._battery_pool_reference_stores[ref_store_key], name, priority
+            pool_ref_store=self._battery_pool_reference_stores[ref_store_key],
+            name=name,
+            priority=priority,
         )
 
     def _data_sourcing_request_sender(self) -> Sender[ComponentMetricRequest]:

src/frequenz/sdk/timeseries/battery_pool/_battery_pool.py

@@ -55,6 +55,7 @@ class BatteryPool:
 
     def __init__(
         self,
+        *,
         pool_ref_store: BatteryPoolReferenceStore,
         name: str | None,
         priority: int,

src/frequenz/sdk/timeseries/ev_charger_pool/_ev_charger_pool.py

@@ -41,6 +41,7 @@ class EVChargerPool:
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         pool_ref_store: EVChargerPoolReferenceStore,
         name: str | None,
         priority: int,

src/frequenz/sdk/timeseries/pv_pool/_pv_pool.py

@@ -34,6 +34,7 @@ class PVPool:
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         pool_ref_store: PVPoolReferenceStore,
         name: str | None,
         priority: int,