grunt-angular-templates icon indicating copy to clipboard operation
grunt-angular-templates copied to clipboard

Put template string in variable before putting in array. This will avoi...

Open ndebellas opened this issue 11 years ago • 4 comments

...d the Maximum call stack size exceeded on mac.

Unfortunate that the spacing has changed. I only intended to change 2 lines. Intent here is to fix an issue with mac where if one of the templates is too big, we get a "Maximum call stack size exceeded".

ndebellas avatar Oct 07 '14 07:10 ndebellas

Let me know if you want me to recommit without the unintended spacing changes... Thank you

ndebellas avatar Oct 07 '14 07:10 ndebellas

I'm not sure about this one, particularly since I haven't heard of the issue nor experienced it.

Any idea on why this was an issue?

ericclemmons avatar Jul 05 '15 00:07 ericclemmons

@ndebellas Is this still a concern? If so I'd suggest re-committing without the spacing so we can more easily grok the changes. I'd also note that I haven't seen anything like this, do you have a sample template that can be used to reproduce?

underscorebrody avatar Nov 26 '15 00:11 underscorebrody

Thank you for reaching out. I'll see if I can recommit without spacing. Also I'll send a template... I believe it's still a concern, but only for large templates...

Nick On Nov 25, 2015 6:46 PM, "eric brody" [email protected] wrote:

@ndebellas https://github.com/ndebellas Is this still a concern? If so I'd suggest re-committing without the spacing so we can more easily grok the changes. I'd also note that I haven't seen anything like this, do you have a sample template that can be used to reproduce?

— Reply to this email directly or view it on GitHub https://github.com/ericclemmons/grunt-angular-templates/pull/108#issuecomment-159768061 .

ndebellas avatar Nov 28 '15 06:11 ndebellas