New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand Literate CoffeeScript to include .litcoffee.md and/or .coffee.md suffixes #2736

Closed
raganwald opened this Issue Feb 27, 2013 · 6 comments

Comments

Projects
None yet
3 participants
@raganwald

raganwald commented Feb 27, 2013

This would make Literate CoffeeScript interoperate with existing markdown tools very smoothly.

@michaelficarra

This comment has been minimized.

Show comment
Hide comment
@michaelficarra

michaelficarra Feb 28, 2013

Collaborator

@databases: markdown is a superset of litcoffee. Litcoffee requires code blocks to be valid coffeescript. Your proposed extension would not be able to support proper highlighting.

Collaborator

michaelficarra commented Feb 28, 2013

@databases: markdown is a superset of litcoffee. Litcoffee requires code blocks to be valid coffeescript. Your proposed extension would not be able to support proper highlighting.

@raganwald

This comment has been minimized.

Show comment
Hide comment
@raganwald

raganwald Feb 28, 2013

I don't understand your comment. I understand that there are Markdown files that are not valid .litcoffee files. But what I'm saying is that if I write something like coffee foo.coffee.md, I want the CoffeeScript interpreter to run it as if it was a valid .litcoffee file. Right now, it assumes that foo.coffee.md is a coffeescript file and borks.

I don't see anything about this proposal that would interfere with proper highlighting, but I am obviously missing something.

raganwald commented Feb 28, 2013

I don't understand your comment. I understand that there are Markdown files that are not valid .litcoffee files. But what I'm saying is that if I write something like coffee foo.coffee.md, I want the CoffeeScript interpreter to run it as if it was a valid .litcoffee file. Right now, it assumes that foo.coffee.md is a coffeescript file and borks.

I don't see anything about this proposal that would interfere with proper highlighting, but I am obviously missing something.

@michaelficarra

This comment has been minimized.

Show comment
Hide comment
@michaelficarra

michaelficarra Feb 28, 2013

Collaborator

Ah, okay. That is reasonable. I misunderstood your proposal.

Collaborator

michaelficarra commented Feb 28, 2013

Ah, okay. That is reasonable. I misunderstood your proposal.

@jashkenas

This comment has been minimized.

Show comment
Hide comment
@jashkenas

jashkenas Feb 28, 2013

Owner

.coffee.md sounds like a fine one to add.

Owner

jashkenas commented Feb 28, 2013

.coffee.md sounds like a fine one to add.

@jashkenas

This comment has been minimized.

Show comment
Hide comment
@jashkenas

jashkenas Feb 28, 2013

Owner

Merged in.

Owner

jashkenas commented Feb 28, 2013

Merged in.

@jashkenas jashkenas closed this Feb 28, 2013

@raganwald

This comment has been minimized.

Show comment
Hide comment
@raganwald

raganwald Mar 2, 2013

Big Thanks!

raganwald commented Mar 2, 2013

Big Thanks!

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