Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Naive CEL validation example #1372

Merged
merged 2 commits into from
Dec 15, 2023
Merged

Naive CEL validation example #1372

merged 2 commits into from
Dec 15, 2023

Conversation

clux
Copy link
Member

@clux clux commented Dec 14, 2023

for #1367 and kube-rs/website#53
was testing it locally against k3d 1.27 (this one is a main run fn and is not tested on CI atm)

for #1367 and kube-rs/website#53

Signed-off-by: clux <sszynrae@gmail.com>
@clux clux added the changelog-exclude changelog excluded prs label Dec 14, 2023
@clux clux added this to the 0.88.0 milestone Dec 14, 2023
Signed-off-by: clux <sszynrae@gmail.com>
Copy link

codecov bot commented Dec 14, 2023

Codecov Report

Merging #1372 (35fc481) into main (289c501) will increase coverage by 0.1%.
Report is 1 commits behind head on main.
The diff coverage is 0.0%.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1372     +/-   ##
=======================================
+ Coverage   72.1%   72.1%   +0.1%     
=======================================
  Files         75      75             
  Lines       6436    6435      -1     
=======================================
  Hits        4635    4635             
+ Misses      1801    1800      -1     
Files Coverage Δ
kube-derive/src/lib.rs 0.0% <ø> (ø)
kube-derive/tests/crd_schema_test.rs 100.0% <ø> (ø)
kube-derive/src/custom_resource.rs 13.7% <0.0%> (+0.1%) ⬆️

@clux clux marked this pull request as ready for review December 14, 2023 04:11
@clux clux merged commit 9dc932b into main Dec 15, 2023
18 checks passed
@clux clux deleted the cel-validation branch December 15, 2023 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-exclude changelog excluded prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant