Box a large runtime error in response to clippy#1880
Conversation
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1880 +/- ##
=======================================
+ Coverage 75.9% 75.9% +0.1%
=======================================
Files 85 85
Lines 8237 8238 +1
=======================================
+ Hits 6244 6248 +4
+ Misses 1993 1990 -3
🚀 New features to boost your workflow:
|
|
Have done a bit of research on this, and it might still break some people, and clarified the body. It still seems like the a good way forward to me, but soliciting some feedback if people have time. EDIT: fixing this in a non-breaking way for now. Probably not worth rushing a breaking change for a small memory optimization. |
Signed-off-by: clux <sszynrae@gmail.com>
nightkr
left a comment
There was a problem hiding this comment.
Ah damn, this would've made sense to get in for 3.0.0. Honestly not sure how to handle this, as-is it's still technically breaking even if it's pretty unlikely to cause much trouble...
|
yeah, i wasn't sure, whether it was a good idea to make a breaking change for it. but having looked more, it's unlikely to be an error users touch often. i'll schedule it for 4.0 |
Fixes a result_large_err clippy lint in runtime by boxing an the
controller::Errorfor #1879. This does not generally break anything for error policies. None of my controllers have been affected. Modified an example to show this user defined error types round trip the same.We generally discourage interrogating the internal reason for reconciliation which is the main thing this error allows you to do. However, people might in advanced use cases hit this, e.g. npmp / k8s-insider-agent from a code search.
Because of this, if we decide this is worth it, it would be a breaking change, and have applied a
changelog-changelabel.