Skip to content

Conversation

@hanrelan
Copy link

This adds support for the empty keyword per spec

I think there's an issue where if a reserved word appears anywhere in the name of a var, it causes a problem. But this seems to occur for nil as well, so I think it's unrelated to this PR

@edgurgel
Copy link
Owner

edgurgel commented Jan 7, 2025

I think there's an issue where if a reserved word appears anywhere in the name of a var, it causes a problem. But this seems to occur for nil as well, so I think it's unrelated to this PR

Yeah you are right! I will create an issue for this. We might need to rename another variable on the integration test for case {{ empty_string | truncatewords: 3, "" }}

@edgurgel edgurgel merged commit 02b9cf8 into edgurgel:main Jan 7, 2025
4 checks passed
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