Sam Pullara
Sam Pullara
I added a test that shows you how to construct a MustacheFactory that has the behavior that you need. This kind of change is one of the reasons that almost...
A few comments (thanks for digging into this): 1. Do you think it makes sense to pass the indent around in the TemplateContext so it doesn't change the callback interface?...
LGTM. Please close the issue when you are convinced it is working for you.
I need to convince myself that it doesn't break backwards compatibility with some wanted behavior.
I'm starting to warmup to just making this the default and removing the old whitespace behavior as it appears to be close enough in running all my current tests. I'd...
Thanks for the report. My guess is that the first solution might work. Taking a loot at it.
I think we can remove it. The differences are pretty small and I don't think it will be disruptive except perhaps for tests which can be pretty easily updated. Right...
Ran into the same problem when I tried to convert your patch. Thinking about it some more as it is really close.
Can you fork then for now? The issue is that if I merge this in I'll have to support this form of it going forward and I would rather have...
It seems reasonable to me to set the input/output encoding on the DefaultMustacheFactory