Update pathval

[koddsson]

Since it's now an ES Module! :)

[43081j]

Good catch! One step closer to pure esm 👀

[voxpelli]

Would have to change from require to import here?

chai/lib/chai/utils/index.js

Line 11 in 1a8247f

var pathval = require('pathval');

[koddsson]

Would have to change from require to import here?

Nope, the file has already got import calls in the 5.x.x branch which this PR is targeting.

[voxpelli]

Missed that this PR was targeting a non-main branch as it wasn't mentioned anywhere and I didn't think of looking at the target branch as its so rare for projects to merge towards a non-main branch.

I would suggest creating a milestone and/or tracking issue that can convey this context to new contributors.

[koddsson]

@keithamus; I don't have merge rights, so I can't land this myself.

It also might be good to remove node12 and node14 as required builds in the repo settings, since they are never going to be reported.

[keithamus]

I've given you merge rights, so feel free to merge 👍