Move away from PEP604 - use of `|` for isinstance/issubclass

[hoxbro]

Problem description

The Ruff rule, which enforced this change, has been removed in the latest version, as it is slower than a tuple. We should therefore revert | back into tuples.

docs.astral.sh/ruff/rules/non-pep604-isinstance

An example of this is here:

bokeh/src/bokeh/core/property/descriptors.py

Line 483 in 6be1a10

return isinstance(value, FunctionType | InstanceDefault)

Feature description

Potential alternatives

Additional information

No response

[mattpap]

Whatever is more readable in Python is also almost always the slower option. Is the difference so significant that we should care about this?

[hoxbro]

I think using a tuple is the more readable option.

[bryevdv]

I think I actually prefer the tuple syntax as well (just familiarity of ~20 years) I don't have a strong opinion though. If someone wants to make a PR for this, I would approve it, and if no one does, I won't mind either.

[maxBRT]

I would like to take care of this. Seems like a good first issue to explore the code base and familiarize myself with it!