Skip to content
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

Coffeefilter #42

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Coffeefilter #42

wants to merge 2 commits into from

Conversation

philoye
Copy link

@philoye philoye commented Nov 7, 2012

Haml 3.2 (currently in beta, almost ready) supports additional filters, one of which is inline coffeescipt. You can read about the beta here.

This PR adds support for syntax highlighting for inline coffeescript.

@tpope
Copy link
Owner

tpope commented Jan 1, 2013

Do you know if the coffeescript runtime files that this depends on will ship with Vim?

@philoye
Copy link
Author

philoye commented Jan 1, 2013

This change depends on coffeescript runtime files? I don't imagine Vim itself would ship with it.

That said, maybe it is the vim-coffee-script plugin that I have installed? Would this break for someone without it?

@tpope
Copy link
Owner

tpope commented May 28, 2013

Probably won't fail, but it might produce a warning in verbose mode. Regardless, there's a low barrier to including runtime files with Vim, so it's pretty silly not to. Now would be a good time to pursue it, with Vim 7.4 imminent.

First things first, though, it doesn't work. Your missing the appropriate syn include.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants