Add support for Cakefiles! #101

Merged
merged 1 commit into from May 29, 2012

Conversation

Projects
None yet
2 participants
Contributor

aseemk commented May 29, 2012

[Repeat of #99 to be based off of dvlp rather than master.]

Discussion thread:

http://groups.google.com/group/streamlinejs/browse_thread/thread/71cabdbc8962b030/afcc631faf404b97

In order to not have to manually reimplement much or all of Cake, this pull request achieves Streamlining via the in-file require('streamline/module') directive.

This commit just recognizes Cakefiles as CoffeeScript so that the generated tmp file is outputted with a .coffee extension, which makes things work.

There might be other ways of supporting Streamline in Cakefiles, in particular to avoid the in-file directive, but I'm not sure any are feasible. Still thinking though.

@aseemk aseemk Add support for Streamlined Cakefiles!
In order to not have to manually reimplement much or all of Cake, achieving
Streamlining via the in-file require('streamline/module') directive.

This change is thus just to recognize Cakefiles as CoffeeScript and output the
generated tmp file with a .coffee extension, which makes things work.
50baa98

@bjouhier bjouhier added a commit that referenced this pull request May 29, 2012

@bjouhier bjouhier Merge pull request #101 from aseemk/cakefiles
Add support for Cakefiles!
938d925

@bjouhier bjouhier merged commit 938d925 into Sage:dvlp May 29, 2012

Contributor

aseemk commented May 29, 2012

Thanks Bruno!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment