Deprecate more complicated way of defining colors.

Carreau · Carreau · commit d9ed43d5e02c · 2024-01-24T21:02:25.000+01:00
Keep simple with a dict.
diff --git a/IPython/utils/coloransi.py b/IPython/utils/coloransi.py
@@ -10,6 +10,7 @@
 
 
 import os
+import warnings
 
 from IPython.utils.ipstruct import Struct
 
@@ -114,8 +115,18 @@ class ColorScheme:
     name: str
     colors: Struct
 
-    def __init__(self,__scheme_name_,colordict=None,**colormap):
+    def __init__(self, __scheme_name_, colordict=None, **colormap):
         self.name = __scheme_name_
+        if colormap:
+            warnings.warn(
+                "Passing each colors as a kwarg to ColorScheme is "
+                "considered for deprecation. Please pass a "
+                "dict as single dict as second parameter. If you are using this"
+                "Feature, please comment an subscribe to issue "
+                "https://github.com/ipython/ipython/issues/14304",
+                PendingDeprecationWarning,
+                stacklevel=2,
+            )
         if colordict is None:
             self.colors = Struct(**colormap)
         else:
