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

Literate CoffeeScript support #243

Closed
wants to merge 4 commits into from
Closed

Literate CoffeeScript support #243

wants to merge 4 commits into from

Conversation

mr-vinn
Copy link
Contributor

@mr-vinn mr-vinn commented Aug 20, 2014

I rebased pull request #173 on the latest master branch to enable Literate CoffeeScript in Tilt 2.x.

Using Literate CoffeeScript in a Rails app

Configure Sprockets

You'll still have to tell Sprockets what to do with .litcoffee files as described by Skalee; a Sprockets developer stated in sstephenson/sprockets#420 that .litcoffee support won't be enabled by default.

Update Gemfile

For now, you also have to update your Gemfile to use the mr-vinn/tilt repo. However, Tilt is locked to 1.4.1 in the latest stable Rails version (4.1.5), so I applied these commits to Tilt 1.4.1 in a different branch. Add this to your Gemfile to get a litcoffee-enabled Tilt 1.4.1:

group :assets do
  gem 'tilt', github: 'mr-vinn/tilt', branch: '1.4.1-litcoffee'
end

Make them DRY before introducing tests for literate CoffeeScript
handler, extract template renderer class to instance variable.
They will be shared with literate CoffeeScript tests.
@judofyr
Copy link
Collaborator

judofyr commented Jul 6, 2015

I'd love to get this in 2.0.2 after I've resolved #264.

@judofyr judofyr added this to the 2.0.2 milestone Jul 6, 2015
@judofyr judofyr added the planned label Jul 7, 2015
judofyr added a commit that referenced this pull request Jul 7, 2015
@judofyr
Copy link
Collaborator

judofyr commented Jul 7, 2015

Thanks! I rebased it again, tweaked some stuff and merged it as d01b743.

@judofyr judofyr closed this Jul 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants