search

paiq / blackcoffee

CoffeeScript + hygienic macros
MIT License
105 stars 9 forks source link

Load file with macros inside #15

Closed hackerhelmut closed 10 years ago

hackerhelmut commented 10 years ago

I've extended the macro.coffee so that it is possible to load files which can include and call macros itself. The macros will be executed in the same context and work on the same utils.

vanviegen commented 10 years ago

Sorry, I don't really understand what these patches are supposed to do. 'load files which can include and call macros itself' is already supported, if I'm understanding you correctly.

Could you perhaps add a unit test for the new behavior? That should make it really clear what behavior has changed.

Thanks!

hackerhelmut commented 10 years ago

Sorry for the delay.

I've tested a bit and extended the macro unit tests in a direction that it might not cover my use case. You're right it is supported. I don't know why I did not see it the time I was writing my macro library.

Unfortunately all unit tests I can come up with are supported but I still struggle to get my library working without any modifications. But as far as I can tell now the problem is on my end. I just need more time to debug.

Best regards Rolf