tektoncd / triggers

Event triggering with Tekton!
Apache License 2.0
545 stars 416 forks source link

Add more cel extensions #1717

Closed venc0r closed 2 months ago

venc0r commented 2 months ago

Feature request

the docs say the cel interceptor should support all extensions

All the functionality from the cel-go project’s CEL extension is available in your CEL expressions.

Over the last months there are new extensions added to cel, but not enabled in the cel-eval interceptor.

Use case

I want to get a slice of an array like: [1,2,3,4].slice(1,3) => [2,3]

khrm commented 2 months ago

@venc0r Won't a update to the latest library fix this? https://github.com/tektoncd/triggers/pull/1718

khrm commented 2 months ago

/assign @khrm

khrm commented 2 months ago

/assign @bigkevmcd

khrm commented 2 months ago

It will be fixed by #1730

bigkevmcd commented 2 months ago

@venc0r #1730 will add most of the new CEL extensions, I deliberately didn't add the Proto extension as I'm unsure of a use case for it, but the slices, sets and math extensions are now in there.