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

docs(README): remove coffeescript from README #337

Merged
merged 1 commit into from
Feb 4, 2016

Conversation

ariporad
Copy link
Contributor

@ariporad ariporad commented Feb 4, 2016

Other than the CoffeeScript section

@nfischer: Ready for review

@@ -68,6 +68,8 @@ if (exec('git commit -am "Auto-commit"').code !== 0) {

### CoffeeScript

CoffeeScript is also supported automatically:s
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think ":s" on the end is a typo

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. I think it is too.

@ariporad ariporad modified the milestones: v0.7.0, v0.6.0 Feb 4, 2016
@ariporad ariporad removed the blocked label Feb 4, 2016
Other than the CoffeeScript section
@ariporad ariporad force-pushed the no-coffeescript-in-README branch from c071ad3 to 7934366 Compare February 4, 2016 23:31
@ariporad
Copy link
Contributor Author

ariporad commented Feb 4, 2016

@nfischer: Ok, everything should be fixed now.

@ariporad ariporad modified the milestones: v0.6.0, v0.7.0 Feb 4, 2016
@nfischer
Copy link
Member

nfischer commented Feb 4, 2016

LGTM

@ariporad
Copy link
Contributor Author

ariporad commented Feb 4, 2016

I'll merge once Travis finishes.

nfischer added a commit that referenced this pull request Feb 4, 2016
docs(README): remove coffeescript from README
@nfischer nfischer merged commit a35f9dd into master Feb 4, 2016
@nfischer nfischer deleted the no-coffeescript-in-README branch February 4, 2016 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants