`path.relative` adds trailing slash in node 12

[dominikwilkowski]

What happens in node < 12

const Path = require('path');
Path.relative('/page1/page2/foo', '/');
// '../../..'
Path.relative('/page1/page2/foo', '/page1/');
// '../..'
Path.relative('/page1/page2/foo', '/page1/page2/');
// '..'

What happens in node => 12

const Path = require('path');
Path.relative('/page1/page2/foo', '/');
// '../../../'
//          ^--- Note the trailing slash
Path.relative('/page1/page2/foo', '/page1/');
// '../..'
Path.relative('/page1/page2/foo', '/page1/page2/');
// '..'

Note the inconsistency when resolving a path against root. In node 12 we now get a trailing slash.

What I expect should happen

We either resolve all paths with or without trailing slashes but not mix both.
Nodes behavior prior to 12 was more consistent.

• 12.6.0
• OSX 10.14.5

[Trott]

The changed behavior appears in Node.js 11.14.0 as well (but not in 11.13.0).

[Trott]

@nodejs/path