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.

Sign up for GitHub

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

Jump to bottom

Add support for compiling literate coffeescript #29

Merged
merged 7 commits into from
Feb 1, 2014
Merged

Add support for compiling literate coffeescript #29

merged 7 commits into from
Feb 1, 2014

Conversation

oychang
Copy link
Contributor

@oychang oychang commented Feb 1, 2014

For those of us who like obscure programming techniques from the 90's, this guard has been seriously lacking. This pull request aims to change that.

Changes:

  • Add watching of .coffee.md and .litcoffee (listed here)
  • Automatically detect need to pass --literate to the compiler
  • Updates tests and docs to reflect changes

@oychang
Add extensions for literate coffeescript watching.
9d72f89
@oychang
Add detection for passing the --literate option.
686bd10
@oychang
Update litcoffee detection regex.
756ba43 
Previously, a.js.coffee -> a.js but b.js.litcoffee -> b.js.js.
Assuming that having double .js is useless, we detect the missing case
properly.

Additionally, update the setting of the literate flag, as
`File.extname` only returns the final extension.
@oychang
Update coffeescript_spec tests to expect literate.
fe70e2b 
Update expected regexes and tests to check .coffee.md and .litcoffee,
with an eye out for improperly escaped period characters.
@oychang
Update inspector_spec to account for literacy.
e31d5de 
Hopefully I'm doing this line wrapping thing right.
@oychang
Update runner_spec to include literate tests.
a3faf54 
Updated old tests to include both new extensions generally, and also add
a new test to ensure the literate option is set.
@oychang
Update README.md to list support for more exts.
fd39e71 
Also, line break on longer lines of options to avoid scrolling in
the fixed width Github readme viewer.
@netzpirat netzpirat merged commit fd39e71 into netzpirat:master Feb 1, 2014
@netzpirat
Copy link
Owner

Very nice pull request! Thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oychang @netzpirat