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

chore: show expr errors #13778

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

tooptoop4
Copy link
Contributor

@tooptoop4 tooptoop4 commented Oct 17, 2024

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 #13774

@agilgur5
Copy link
Contributor

co-authored by @boiledfroginthewell in #13774

Please add them as co-author in the commit itself as I mentioned in #13745 (comment).

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 #13759 (comment)

@agilgur5 agilgur5 changed the title chore: show expr errors for #13759 chore: show expr errors Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants