search

argoproj / argo-workflows

Workflow Engine for Kubernetes
https://argo-workflows.readthedocs.io/
Apache License 2.0
15.11k stars 3.21k forks source link

chore: show `expr` errors #13778

Open tooptoop4 opened 1 month ago

tooptoop4 commented 1 month ago

For #13759 and #11101

Logs error details for expr. ie 

failed to compile:  unexpected token Operator(".") (1:28)
| sprig.trim(" abc ") | sprig.upper()
| ...........................^

co-authored by @boiledfroginthewell in https://github.com/argoproj/argo-workflows/pull/13774

agilgur5 commented 1 month ago

co-authored by @boiledfroginthewell in #13774

Please add them as co-author in the commit itself as I mentioned in https://github.com/argoproj/argo-workflows/pull/13745#issuecomment-2408415438.

In this case, the PR was only closed a few hours ago, so I would ask the original author to re-open rather. They also gave a rationale for closing in https://github.com/argoproj/argo-workflows/issues/13759#issuecomment-2419549947