Skip to content

Commit

Permalink
refactor(require): Removed support for passing a function to appRootP…
Browse files Browse the repository at this point in the history
…ath.require()

BREAKING CHANGE: The provided require() method no longer supports passing require into it (not

commonly used, deprecated for a while)
  • Loading branch information
inxilpro committed Sep 7, 2016
1 parent 6098bfd commit dc3eaa5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
9 changes: 0 additions & 9 deletions lib/app-root-path.js
Expand Up @@ -11,15 +11,6 @@ module.exports = function(dirname) {
},

require: function(pathToModule) {
// Backwards compatibility check
if ('function' === typeof pathToModule) {
console.warn('Just use appRootPath.require() -- no need to pass in your ' +
'modules\'s require() function any more.');
return function(pathToModule) {
return publicInterface.require(pathToModule);
}
}

return require(publicInterface.resolve(pathToModule));
},

Expand Down
4 changes: 2 additions & 2 deletions package.json
Expand Up @@ -7,7 +7,7 @@
"scripts": {
"test": "nyc mocha -R spec",
"report-coverage": "npm test && nyc report --reporter=text-lcov > coverage.lcov && codecov",
"release": "semantic-release pre && npm publish --tag next && semantic-release post"
"release": "semantic-release pre && npm publish && semantic-release post"
},
"repository": {
"type": "git",
Expand Down Expand Up @@ -48,7 +48,7 @@
"node": ">= 4.0.0"
},
"release": {
"branch": "development"
"branch": "master"
},
"config": {
"ghooks": {
Expand Down

0 comments on commit dc3eaa5

Please sign in to comment.