Skip to content

Commit

Permalink
Merge pull request #18 from nemanjan00/1.x
Browse files Browse the repository at this point in the history
Dependency tree support
  • Loading branch information
bazilio91 committed Jun 9, 2016
2 parents 220a457 + 499d1e3 commit ac32d8b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ module.exports = function (source) {
this.cacheable && this.cacheable();
var template = ejs.compile(source, {
client: true,
filename: '.'
filename: '.',
webpack: this
});

var ast = uglify.parser.parse(template.toString());
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
},
"repository": {
"type": "git",
"url": "https://github.com/bazilio91/ejs-compiled-loader.git"
"url": "https://github.com/bazilio91/ejs-compiled-loader.git#ba8ef7f"
},
"keywords": [
"ejs",
Expand All @@ -23,7 +23,7 @@
},
"homepage": "https://github.com/bazilio91/ejs-compiled-loader",
"dependencies": {
"ejs": "^1.0.0",
"ejs": "git+https://github.com/nemanjan00/ejs.git",

This comment has been minimized.

Copy link
@vanslly

vanslly Jun 13, 2017

@bazilio91 this is causing npm to not install the ejs package for me. I'm on [email protected].

This comment has been minimized.

Copy link
@bazilio91

bazilio91 Jun 13, 2017

Author Owner

Just tested

~/test » npm install [email protected]                                                                     
npm WARN [email protected] No description

+ [email protected]
added 3 packages in 1.455s
~/test » npm -v
5.0.3
~/test » node -v
v6.8.1

Can you supply more info?

This comment has been minimized.

Copy link
@kbtz

kbtz Jun 13, 2017

@vanslly May you try using the package for the 2.x version?
https://www.npmjs.com/package/ejs-render-loader

This comment has been minimized.

Copy link
@vanslly

vanslly Jun 13, 2017

I will test again and also try 2.x. Btw, I am on node 7.9.0. Is having a git ref supported by npm? I thought that's a jspm thing

This comment has been minimized.

Copy link
@vanslly

This comment has been minimized.

Copy link
@vanslly

vanslly Jun 13, 2017

OK.. this is working when I don't use sinopia as my npm proxy

"uglify-js": "~1.3.4"
}
}

0 comments on commit ac32d8b

Please sign in to comment.