File Formats will be supported coffee-script #8

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@hisayan
Contributor
hisayan commented Jan 4, 2012

I usually use the coffeescript.
It is very easy to implement.

@lorenwest lorenwest and 1 other commented on an outdated diff Jan 4, 2012
package.json
@@ -8,7 +8,8 @@
"directories": {"lib": "./lib", "config": "./config", "test": "./test"},
"dependencies": {
"yaml" : "0.2.x",
- "vows" : "0.5.x"
+ "vows" : "0.5.x",
+ "coffee-script" : "~1.2.0"
@lorenwest
lorenwest Jan 4, 2012 Owner

I'm curious - what's the ~ (tilde) syntax mean?

@lorenwest lorenwest commented on the diff Jan 4, 2012
lib/config.js
@@ -2,6 +2,7 @@
var Sys = require('sys'),
File = require('fs'),
Yaml = require('yaml'),
+ Coffee = require('coffee-script'),
@lorenwest
lorenwest Jan 4, 2012 Owner

Thank you for adding this format. The pull request looks good.

In order to merge the request and publish a new branch it will need to be accompanied with a test. If you would like to write it, then I'll publish as soon as it's ready. Otherwise I can write it, but it'll be a week or two before I can get to it.

It's actually very easy. Create a .coffee configuration and test it in test/2-config-test.js. You can see how other formats are tested in there.

The current branch is v0.4.9. I will create it in github so you can attach the pull request to that branch.

Thanks,
-Loren

@hisayan
hisayan Jan 5, 2012 Contributor

Thanks.

I want to try writing configuration and test in few days.
and work at current branch.

my English is not good.

@lorenwest lorenwest closed this Jan 6, 2012
@lorenwest
Owner

This work moved to a different pull request.

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