breaking changes for Trixi.jl v0.5

[ranocha]

This includes a first set of breaking changes for Trixi.jl v0.5 described in #938.

@jlchan Could you please check that I removed only the DGMulti code that you intended to be removed?

I will leave this in draft mode until we are ready for the breaking release of Trixi.jl. We also need to update the compat bounds of Trixi.jl at Trixi2Vtk.jl to create our documentation here.

[jlchan]

@ranocha thanks for organizing this.

For DGMulti, it would be nice to make the DGMultiMesh constructors consistent so that dg is the first argument. This would be a breaking change. If there is some time before v0.5, I can take care of that sometime next week.

[ranocha]

@ranocha thanks for organizing this.

For DGMulti, it would be nice to make the DGMultiMesh constructors consistent so that dg is the first argument. This would be a breaking change. If there is some time before v0.5, I can take care of that sometime next week.

That sounds good to me - it would be great if you could introduce the corresponding deprecations next week 👍

[jlchan]

@ranocha I added the deprecations, along with some internal cleanup.

I removed a constructor in 2a0205d which is, as far as I can tell, unused. I wasn't sure how to deprecate this properly - do we need to?

[ranocha]

I added the deprecations, along with some internal cleanup.

Thanks!

I removed a constructor in 2a0205d which is, as far as I can tell, unused. I wasn't sure how to deprecate this properly - do we need to?

No, I think its okay. Was it documented, e.g., via a docstring? If so, it would be great if you could mention it in NEWS.md.

[jlchan]

No docstring. I'll add it to the news file soon

[ranocha]

Thanks!

[sloede]

Two small comments, otherwise this LGTM!

[sloede]

🎉