use [[deprecated("")]] for deprecated function instead of static assert?
In [porousmediumflow/nonisothermal/localresidual.hh](https://git.iws.uni-stuttgart.de/dumux-repositories/dumux/-/blob/master/dumux/porousmediumflow/nonisothermal/localresidual.hh?ref_type=heads), the function `static void fluidPhaseStorage(NumEqVector& storage,const SubControlVolume& scv,const VolumeVariables& volVars,int phaseIdx)` (line [50](https://git.iws.uni-stuttgart.de/dumux-repositories/dumux/-/blob/master/dumux/porousmediumflow/nonisothermal/localresidual.hh?ref_type=heads#L50) and [153](https://git.iws.uni-stuttgart.de/dumux-repositories/dumux/-/blob/master/dumux/porousmediumflow/nonisothermal/localresidual.hh?ref_type=heads#L153)) is deprecated as the new one should be called with the additional argument `const Problem& problem`. The deprecation is marked by a `static_assert` inside this function. Shouldn't it be instead marked with `[[deprecated("")]]` to receive a warning, when the old function is used? As the old function does not compute anything and the static_assert has no condition (just the string), the assert is not even failing. Therefore, it is very hard to detect, when this old function is still used.
issue