Closed kbrock closed 1 year ago
@bdunne @Fryguy I mostly wanted to document it as we were greping our code base.
both versions of this method have been around since 2016. Since these methods are only used by customer automate scripts running in production, I thought it didn't make sense to introduce the deprecation messages.
I'm also not sure if we want to push supports_resize
or supports?(:resize)
, or just let them all slide.
refs:
update:
I still feel it is best to just add this as a reference for our future selves and not deprecate or make any changes here.
Since these methods are only used by customer automate scripts running in production, I thought it didn't make sense to introduce the deprecation messages.
That's exactly why we want to mark it deprecated - at least they should see the deprecation in the logs, and then we can remove it later.
But I agree that this PR is just to document and we can deprecate in a separate PR.
We have
validate_resize
for backwards compatibility. It feels safer to just implement this rather than ask the customer to find this method and update it.the 2 validate methods look like they are implementing the AvailabilityMixin but they are just a boolean method instead