Update pre-commit (#6867)

.pre-commit-config.yaml

@@ -4,7 +4,7 @@
 default_stages: [commit]
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v4.6.0
     hooks:
       - id: check-builtin-literals
       - id: check-case-conflict
@@ -17,7 +17,7 @@ repos:
         exclude: \.min\.js$
       - id: trailing-whitespace
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.9
+    rev: v0.4.5
     hooks:
       - id: ruff
         files: panel/
@@ -33,13 +33,13 @@ repos:
         args: [-i, tags]
         exclude: '^lite/files/.*/'
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.6
+    rev: v2.3.0
     hooks:
       - id: codespell
         additional_dependencies:
           - tomli
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v4.0.0-alpha.8
+  - repo: https://github.com/hoxbro/prettier-pre-commit
+    rev: v3.2.5
     hooks:
       - id: prettier
         entry: prettier --write --ignore-unknown --no-error-on-unmatched-pattern
@@ -50,7 +50,7 @@ repos:
       - id: oxipng
         stages: [manual]
   - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: v8.56.0
+    rev: v9.3.0
     hooks:
       - id: eslint
         args: ['-c', 'panel/.eslintrc.js', 'panel/*.ts', 'panel/models/**/*.ts']

CODE_OF_CONDUCT.md

@@ -13,7 +13,7 @@ If you are a leading community member: **Please act as a role model**.
 We as members, contributors, and leaders pledge to make participation in our
 community a harassment-free experience for everyone, regardless of age, body
 size, visible or invisible disability, ethnicity, sex characteristics, gender
-identity and expression, level of experience, education, socio-economic status,
+identity and expression, level of experience, education, socioeconomic status,
 nationality, personal appearance, race, caste, color, religion, or sexual
 identity and orientation.
 

examples/reference/widgets/ToggleGroup.ipynb

@@ -122,7 +122,7 @@
    "source": [
     "### Controls\n",
     "\n",
-    "The `ToggleGroup` widget (concretly the created, underlying widget) exposes a number of options which can be changed from both Python and Javascript. Try out the effect of these parameters interactively:"
+    "The `ToggleGroup` widget (concretely the created, underlying widget) exposes a number of options which can be changed from both Python and Javascript. Try out the effect of these parameters interactively:"
    ]
   },
   {

panel/command/__init__.py

@@ -79,7 +79,7 @@ def main(args=None):
 
     if len(sys.argv) == 1:
         all_commands = sorted([c.name for c in bokeh_commands])
-        die("ERROR: Must specify subcommand, one of: %s" % nice_join(all_commands))
+        die(f"ERROR: Must specify subcommand, one of: {nice_join(all_commands)}")
 
     if sys.argv[1] in ('--help', '-h'):
         args = parser.parse_args(sys.argv[1:])

panel/command/serve.py

@@ -343,7 +343,7 @@ def customize_kwargs(self, args, server_kwargs):
             pattern = REST_PROVIDERS[args.rest_provider](files, args.rest_endpoint)
             patterns.extend(pattern)
         elif args.rest_provider is not None:
-            raise ValueError("rest-provider %r not recognized." % args.rest_provider)
+            raise ValueError(f"rest-provider {args.rest_provider!r} not recognized.")
 
         config.autoreload = args.autoreload
         config.global_loading_spinner = args.global_loading_spinner
@@ -379,7 +379,7 @@ def customize_kwargs(self, args, server_kwargs):
         if args.liveness:
             argvs = {f: args.args for f in files}
             applications = build_single_handler_applications(files, argvs)
-            patterns += [(r"/%s" % args.liveness_endpoint, LivenessHandler, dict(applications=applications))]
+            patterns += [(rf"/{args.liveness_endpoint}", LivenessHandler, dict(applications=applications))]
 
         config.profiler = args.profiler
         if args.admin:

panel/config.py

@@ -746,8 +746,8 @@ def __call__(self, *args, **params):
                     state._extensions.append(arg)
                 ReactiveHTMLMetaclass._loaded_extensions.add(arg)
             else:
-                self.param.warning('%s extension not recognized and '
-                                   'will be skipped.' % arg)
+                self.param.warning(f'{arg} extension not recognized and '
+                                   'will be skipped.')
 
         for k, v in params.items():
             if k == 'design' and isinstance(v, str):

panel/interact.py

@@ -109,7 +109,7 @@ def __init__(self, object, params={}, **kwargs):
         self._inner_layout = Row(self._pane)
         widgets = [widget for _, widget in widgets if isinstance(widget, Widget)]
         if 'name' in params:
-            widgets.insert(0, HTML('<h2>%s</h2>' % self.name))
+            widgets.insert(0, HTML(f'<h2>{self.name}</h2>'))
         self.widget_box = Column(*widgets)
         self.layout.objects = [self.widget_box, self._inner_layout]
         self._link_widgets()

panel/io/admin.py

@@ -257,7 +257,7 @@ def update_cds(new, nb=False):
         else:
             msg = new.getMessage()
             line_color = 'black'
-            if msg.startswith('Session %s logged' % sid):
+            if msg.startswith(f'Session {sid} logged'):
                 etype = 'logging'
                 line_color = EVENT_TYPES.get(etype)
             elif msg.startswith(LOG_SESSION_DESTROYED % sid):

panel/io/model.py

@@ -157,7 +157,7 @@ def bokeh_repr(obj: Model, depth: int = 0, ignored: Optional[Iterable[str]] = No
         if k in ignored:
             continue
         if isinstance(v, Model):
-            v = '%s()' % type(v).__name__
+            v = f'{type(v).__name__}()'
         else:
             v = repr(v)
         if len(v) > 30:
@@ -168,7 +168,7 @@ def bokeh_repr(obj: Model, depth: int = 0, ignored: Optional[Iterable[str]] = No
         r += f'{cls}(children=[\n'
         for child_obj in obj.children: # type: ignore
             r += textwrap.indent(bokeh_repr(child_obj, depth=depth+1) + ',\n', '  ')
-        r += '], %s)' % props_repr
+        r += f'], {props_repr})'
     else:
         r += f'{cls}({props_repr})'
     return r

panel/io/rest.py

@@ -122,7 +122,7 @@ def tranquilizer_rest_provider(files, endpoint):
     """
     app = build_tranquilize_application(files)
     tr = WSGIContainer(app)
-    return [(r"^/%s/.*" % endpoint, web.FallbackHandler, dict(fallback=tr))]
+    return [(rf"^/{endpoint}/.*", web.FallbackHandler, dict(fallback=tr))]
 
 
 def param_rest_provider(files, endpoint):
@@ -169,7 +169,7 @@ def param_rest_provider(files, endpoint):
 
     if endpoint and not endpoint.endswith('/'):
         endpoint += '/'
-    return [((r"^/%s.*" % endpoint if endpoint else r"^.*"), ParamHandler, dict(root=endpoint))]
+    return [((rf"^/{endpoint}.*" if endpoint else r"^.*"), ParamHandler, dict(root=endpoint))]
 
 
 REST_PROVIDERS = {

panel/io/save.py

@@ -243,8 +243,8 @@ def save(
             resources = INLINE
             mode = 'inline'
         else:
-            raise ValueError("Resources %r not recognized, specify one "
-                             "of 'CDN' or 'INLINE'." % resources)
+            raise ValueError(f"Resources {resources!r} not recognized, specify one "
+                             "of 'CDN' or 'INLINE'.")
     elif isinstance(resources, BkResources):
         mode = resources.mode
 

panel/io/server.py

@@ -857,9 +857,9 @@ def get_static_routes(static_dirs):
                              "this is reserved for internal use.")
         path = fullpath(path)
         if not os.path.isdir(path):
-            raise ValueError("Cannot serve non-existent path %s" % path)
+            raise ValueError(f"Cannot serve non-existent path {path}")
         patterns.append(
-            (r"%s/(.*)" % slug, StaticFileHandler, {"path": path})
+            (rf"{slug}/(.*)", StaticFileHandler, {"path": path})
         )
     patterns.append((
         f'/{COMPONENT_PATH}(.*)', ComponentResourceHandler, {}
@@ -1085,7 +1085,7 @@ def get_server(
 
     if liveness:
         liveness_endpoint = 'liveness' if isinstance(liveness, bool) else liveness
-        extra_patterns += [(r"/%s" % liveness_endpoint, LivenessHandler, dict(applications=apps))]
+        extra_patterns += [(rf"/{liveness_endpoint}", LivenessHandler, dict(applications=apps))]
 
     opts = dict(kwargs)
     if loop:

panel/layout/base.py

@@ -81,7 +81,7 @@ def __repr__(self, depth: int = 0, max_depth: int = 10) -> str:
             template = '{cls}({params}){spacer}{objs}'
         return template.format(
             cls=cls, params=', '.join(params),
-            objs=('%s' % spacer).join(objs), spacer=spacer)
+            objs=str(spacer).join(objs), spacer=spacer)
 
     #----------------------------------------------------------------
     # Callback API
@@ -435,9 +435,8 @@ def clone(self, *objects: Any, **params: Any) -> 'ListLike':
             else:
                 objects = self.objects
         elif 'objects' in params:
-            raise ValueError("A %s's objects should be supplied either "
-                             "as arguments or as a keyword, not both."
-                             % type(self).__name__)
+            raise ValueError(f"A {type(self).__name__}'s objects should be supplied either "
+                             "as arguments or as a keyword, not both.")
         p = dict(self.param.values(), **params)
         del p['objects']
         return type(self)(*objects, **p)
@@ -552,9 +551,9 @@ class NamedListLike(param.Parameterized):
     def __init__(self, *items: list[Any | tuple[str, Any]], **params: Any):
         if 'objects' in params:
             if items:
-                raise ValueError('%s objects should be supplied either '
+                raise ValueError(f'{type(self).__name__} objects should be supplied either '
                                  'as positional arguments or as a keyword, '
-                                 'not both.' % type(self).__name__)
+                                 'not both.')
             items = params.pop('objects')
         params['objects'], self._names = self._to_objects_and_names(items)
         super().__init__(**params)
@@ -814,9 +813,9 @@ def __init__(self, *objects: Any, **params: Any):
         from ..pane import panel
         if objects:
             if 'objects' in params:
-                raise ValueError("A %s's objects should be supplied either "
+                raise ValueError(f"A {type(self).__name__}'s objects should be supplied either "
                                  "as positional arguments or as a keyword, "
-                                 "not both." % type(self).__name__)
+                                 "not both.")
             params['objects'] = [panel(pane) for pane in objects]
         elif 'objects' in params:
             objects = params['objects']

panel/layout/flex.py

@@ -71,9 +71,9 @@ def __init__(self, *objects, **params):
                 params['sizing_mode'] = 'stretch_height'
         if objects:
             if 'objects' in params:
-                raise ValueError("A %s's objects should be supplied either "
+                raise ValueError(f"A {type(self).__name__}'s objects should be supplied either "
                                  "as positional arguments or as a keyword, "
-                                 "not both." % type(self).__name__)
+                                 "not both.")
             params['objects'] = [panel(pane) for pane in objects]
         elif 'objects' in params:
             objects = params['objects']

panel/links.py

@@ -77,7 +77,7 @@ def assert_target_syncable(
         elif p not in target.param and p not in list(target._rename.values()):
             matches = difflib.get_close_matches(p, list(target.param))
             if matches:
-                matches_repr = ' Similar parameters include: %r' % matches
+                matches_repr = f' Similar parameters include: {matches!r}'
             else:
                 matches_repr = ''
             raise ValueError(
@@ -146,7 +146,7 @@ def __init__(
            a JS code snippet to execute.
         """
         if source is None:
-            raise ValueError('%s must define a source' % type(self).__name__)
+            raise ValueError(f'{type(self).__name__} must define a source')
         # Source is stored as a weakref to allow it to be garbage collected
         self._source = None if source is None else weakref.ref(source)
         if not args:
@@ -289,7 +289,7 @@ class Link(Callback):
 
     def __init__(self, source: 'Reactive', target: Optional['JSLinkTarget'] = None, **params):
         if self._requires_target and target is None:
-            raise ValueError('%s must define a target.' % type(self).__name__)
+            raise ValueError(f'{type(self).__name__} must define a target.')
         # Source is stored as a weakref to allow it to be garbage collected
         self._target = None if target is None else weakref.ref(target)
         super().__init__(source, **params)
@@ -709,8 +709,7 @@ def _initialize_models(
                 setattr(tgt_model, tgt_spec, value)
         if tgt_model is None and not link.code:
             raise ValueError('Model could not be resolved on target '
-                             '%s and no custom code was specified.' %
-                             type(self.target).__name__)
+                             f'{type(self.target).__name__} and no custom code was specified.')
 
     def _process_references(self, references: dict[str, str]) -> None:
         """

panel/models/reactive_html.py

@@ -105,17 +105,17 @@ def handle_data(self, data):
             if match[2:-1] not in self.loop_var_map[var]:
                 self.loop_var_map[var].append(match[2:-1])
             if var.endswith('.index0'):
-                matches.append('${%s }}]}' % var)
+                matches.append('${%s }}]}' % var)  # noqa: UP031
             else:
-                matches.append('${%s}' % var)
+                matches.append('${%s}' % var)  # noqa: UP031
 
         literal_matches = []
         for match in self._literal_re.findall(data):
             match = match[2:-2].strip()
             if match.endswith('.index0'):
-                literal_matches.append('{{%s }}]}' % match)
+                literal_matches.append('{{%s }}]}' % match)  # noqa: UP031
             else:
-                literal_matches.append('{{ %s }}' % match)
+                literal_matches.append('{{ %s }}' % match)  # noqa: UP031
 
         # Detect templating for loops
         list_loop = re.findall(list_iter_re, data)
@@ -175,7 +175,7 @@ def handle_data(self, data):
         if match and (match.strip() in self.loop_map or '[' in match) and self._open_for:
             if match.strip() in self.loop_map:
                 loop_match = self.loop_map[match.strip()]
-                matches[matches.index('${%s}' % match)] = '${%s}' % loop_match
+                matches[matches.index(f'${{{match}}}')] = f'${{{loop_match}}}'
                 match = loop_match
             elif '[' in match:
                 match, _ = match.split('[')

panel/pane/base.py

@@ -454,8 +454,8 @@ def get_pane_type(cls, obj: Any, **kwargs) -> type['PaneBase']:
                 raise ValueError('If a Pane declares no priority '
                                  'the applies method should return a '
                                  'priority value specific to the '
-                                 'object type or False, but the %s pane '
-                                 'declares no priority.' % p.__name__)
+                                 f'object type or False, but the {p.__name__} pane '
+                                 'declares no priority.')
             elif priority is None or priority is False:
                 continue
             descendents.append((priority, applies, p))
@@ -469,7 +469,7 @@ def get_pane_type(cls, obj: Any, **kwargs) -> type['PaneBase']:
             if not applies:
                 continue
             return pane_type
-        raise TypeError('%s type could not be rendered.' % type(obj).__name__)
+        raise TypeError(f'{type(obj).__name__} type could not be rendered.')
 
 
 class ModelPane(PaneBase):

panel/pane/holoviews.py

@@ -591,7 +591,7 @@ def widgets_from_dimensions(cls, object, widget_types=None, widgets_type='indivi
             object = object.hmap
 
         if isinstance(object, DynamicMap) and object.unbounded:
-            dims = ', '.join('%r' % dim for dim in object.unbounded)
+            dims = ', '.join(f'{dim!r}' for dim in object.unbounded)
             msg = ('DynamicMap cannot be displayed without explicit indexing '
                    'as {dims} dimension(s) are unbounded. '
                    '\nSet dimensions bounds with the DynamicMap redim.range '

panel/pane/image.py

@@ -425,8 +425,8 @@ def applies(cls, obj: Any) -> float | bool | None:
 
     def _type_error(self, object):
         if isinstance(object, str):
-            raise ValueError("%s pane cannot parse string that is not a filename, "
-                             "URL or a SVG XML contents." % type(self).__name__)
+            raise ValueError(f"{type(self).__name__} pane cannot parse string that is not a filename, "
+                             "URL or a SVG XML contents.")
         super()._type_error(object)
 
     def _data(self, obj):

panel/pane/vtk/synchronizable_deserializer.py

@@ -31,7 +31,7 @@ def capitalize(name):
 def fill_array(vtk_arr, state, zf):
     vtk_arr.SetNumberOfComponents(state['numberOfComponents'])
     vtk_arr.SetNumberOfTuples(state['size']//state['numberOfComponents'])
-    data = zf.read('data/%s' % state['hash'])
+    data = zf.read('data/{}'.format(state['hash']))
     dataType = arrayTypesMapping[vtk_arr.GetDataType()]
     elementSize = struct.calcsize(dataType)
     if vtk_arr.GetDataType() == 12:

panel/pane/vtk/synchronizable_serializer.py

@@ -71,7 +71,7 @@ def getJSArrayType(dataArray):
 def zipCompression(name, data):
     with io.BytesIO() as in_memory:
         with zipfile.ZipFile(in_memory, mode="w") as zf:
-            zf.writestr('data/%s' % name,
+            zf.writestr(f'data/{name}',
                         data, zipfile.ZIP_DEFLATED)
         in_memory.seek(0)
         return in_memory.read()
@@ -393,7 +393,7 @@ def pad(depth):
 
 
 def wrapId(idStr):
-    return 'instance:${%s}' % idStr
+    return f'instance:${{{idStr}}}'
 
 # -----------------------------------------------------------------------------
 
@@ -570,7 +570,7 @@ def extractRequiredFields(extractedFields, parent, dataset, context, requestedFi
 
 def annotationSerializer(parent, prop, propId, context, depth):
     if context.debugSerializers:
-        print('%s!!!Annotations are not handled directly by vtk.js but by bokeh model' % pad(depth))
+        print(f'{pad(depth)}!!!Annotations are not handled directly by vtk.js but by bokeh model')
 
     context.addAnnotation(parent, prop, propId)
 
@@ -1243,8 +1243,7 @@ def rendererSerializer(parent, instance, objId, context, depth):
             dependencies.append(viewPropInstance)
             viewPropIds.append(viewPropId)
 
-    calls += context.buildDependencyCallList('%s-props' %
-                                             objId, viewPropIds, 'addViewProp', 'removeViewProp')
+    calls += context.buildDependencyCallList(f'{objId}-props', viewPropIds, 'addViewProp', 'removeViewProp')
 
     return {
         'parent': context.getReferenceId(parent),