Skip to content

Commit 6fc5e95

Browse files
targossilverwind
authored andcommitted
module: allow require('.')
Previously, the minimal argument to require the current directory was require('./'). This commits allows to skip the trailing slash. Fixes: #1178 PR-URL: #1185 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By: Christian Tellnes <christian@tellnes.no> Reviewed-By: Roman Reiss <me@silverwind.io>
1 parent a0d32ff commit 6fc5e95

File tree

5 files changed

+10
-1
lines changed

5 files changed

+10
-1
lines changed

lib/module.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ Module._resolveLookupPaths = function(request, parent) {
205205
}
206206

207207
var start = request.substring(0, 2);
208-
if (start !== './' && start !== '..') {
208+
if (start !== '.' && start !== './' && start !== '..') {
209209
var paths = modulePaths;
210210
if (parent) {
211211
if (!parent.paths) parent.paths = [];
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
module.exports = require('./');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
module.exports = require('.');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
exports.value = 42;

test/parallel/test-require-extensions-main.js

+6
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,9 @@ var common = require('../common');
22
var assert = require('assert');
33

44
require(common.fixturesDir + '/require-bin/bin/req.js');
5+
6+
var a = require(common.fixturesDir + '/module-require/relative/dot.js');
7+
var b = require(common.fixturesDir + '/module-require/relative/dot-slash.js');
8+
9+
assert.equal(a.value, 42);
10+
assert.equal(a, b, 'require(".") should resolve like require("./")');

0 commit comments

Comments
 (0)