Skip to content

Fixed coffeescript syntax in top example - #99

Merged
arturadib merged 1 commit into
shelljs:masterfrom
maxnordlund:patch-1
Aug 29, 2015
Merged

arturadib merged 1 commit into
shelljs:masterfrom
maxnordlund:patch-1

Conversation

@maxnordlund

Copy link
Copy Markdown
Contributor

The cat 'macro.js' needs parentesis to not include file. The current coffeescript compiles to the following:

sed('-i', /.*REPLACE_LINE_WITH_MACRO.*\n/, cat('macro.js', file));

The `cat 'macro.js'` needs parentesis to not include `file`.

```javascript
sed('-i', /.*REPLACE_LINE_WITH_MACRO.*\n/, cat('macro.js', file));
```
arturadib added a commit that referenced this pull request Aug 29, 2015
Fixed coffeescript syntax in top example
@arturadib
arturadib merged commit 3fef967 into shelljs:master Aug 29, 2015
@maxnordlund
maxnordlund deleted the patch-1 branch August 29, 2015 20:03
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