Make `check_on_set` static and `True` by default

holoviz / param

Param: Make your Python code clearer and more reliable by declaring Parameters

https://param.holoviz.org

BSD 3-Clause "New" or "Revised" License

412 stars 69 forks source link

Make `check_on_set` static and `True` by default #835

Closed maximlt closed 9 months ago

maximlt commented 9 months ago

This was possible with only making a few changes to the test suite. However, removing _update_state proves to be less straightforward, I'll keep looking tomorrow.

maximlt commented 9 months ago

Running this branch against Panel's unit tests suite shows a number of errors, which are a hint that this change cannot be made without going through a normal deprecation period.

jbednar commented 9 months ago

Ok, let's include a statement in the Param 2.0 release notes that setting check_on_set=False explicitly is now required in Param 2.0 in any case where the user wants it to be False, and mention that later releases will warn and eventually disallow it.