search

kube-rs / kube

Rust Kubernetes client and controller runtime
https://kube.rs
Apache License 2.0
3.05k stars 318 forks source link

update jsonptr + json-patch #1600

Closed aviramha closed 1 month ago

aviramha commented 1 month ago

json-patch has v3 so can update jsonptr. I also made the requirement less specific, to allow friendlier downstream use (can understand if that's not desired though) Closes https://github.com/kube-rs/kube/issues/1563

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 75.2%. Comparing base (2280962) to head (04d8ada). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
kube-runtime/src/finalizer.rs 0.0% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1600 +/- ## ===================================== Coverage 75.2% 75.2% ===================================== Files 82 82 Lines 7336 7336 ===================================== Hits 5514 5514 Misses 1822 1822 ``` | [Files with missing lines](https://app.codecov.io/gh/kube-rs/kube/pull/1600?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kube-rs) | Coverage Δ | | |---|---|---| | [kube-core/src/admission.rs](https://app.codecov.io/gh/kube-rs/kube/pull/1600?src=pr&el=tree&filepath=kube-core%2Fsrc%2Fadmission.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kube-rs#diff-a3ViZS1jb3JlL3NyYy9hZG1pc3Npb24ucnM=) | `61.2% <ø> (ø)` | | | [kube-runtime/src/finalizer.rs](https://app.codecov.io/gh/kube-rs/kube/pull/1600?src=pr&el=tree&filepath=kube-runtime%2Fsrc%2Ffinalizer.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kube-rs#diff-a3ViZS1ydW50aW1lL3NyYy9maW5hbGl6ZXIucnM=) | `0.0% <0.0%> (ø)` | |