populse / soma-base

Miscelaneous all-purpose classes and functions
0 stars 3 forks source link

in python >= 3.9, use list[] instead. #75

Open DimitriPapadopoulos opened 8 months ago

DimitriPapadopoulos commented 8 months ago

https://github.com/populse/soma-base/blob/6915c1e14327d8af6c9c21e723e904879d4e7a27/python/soma/controller/field.py#L671

Branch 6.0 supports Python ≥ 3.9, so get rid of this.

sapetnioc commented 8 months ago

This is to be backward-compatible with existing code (outside soma-base) still using this syntax. I do not know if there is still such code. Not impossible.

DimitriPapadopoulos commented 8 months ago

Does this mean that this cannot ever be changed? Or that we just need to wait for a few years for existing code to be fixed? How to reach dependees so that they fix their code?