Deprecate `inplace` arguments of `crop` and `reproject`

[rhugonnet]

Having an inplace behaviour related to the data array is bad for serialization, and not recommended (cf Xarray behaviour that always returns a new object, and recommend using ds = ds.isel).

To make things consistent with the Xarray accessor, we'll deprecate inplace for crop, icrop and reproject.