revert compatibility

[lamweili]

In commit 9f62693, let was used.

word-wrap/index.js

Lines 9 to 10 in 207044e

	let lastCharPos = str.length - 1;
	let lastChar = str[lastCharPos];

However, the package.json specifies "node": ">=0.10.0" compatibility.

word-wrap/package.json

Line 28 in 207044e

"node": ">=0.10.0"

This resulted in the following errors for older Node.js:

SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode

Thus, we should change let to var to maintain compatibility for ^1.x.x in adherence to semver.
If compatibility needs to be upgraded, it should be the next major semver (^2.x.x) release.

[doowb]

Thanks @lamweili . Should const also be replaced?

[lamweili]

@doowb, const is good according to node.green.
My GitHub actions, which test older Node.js (v4) for compatibility issues, didn't flag any errors.

[DmitryZ-outten]

@lamweili Thanks! Have the same issue with node v4

[bahamat]

I'm having a problem with let on node 4 as well:

21:12:26    let lastCharPos = str.length - 1;
21:12:26    ^^^
21:12:26  
21:12:26  SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode

If I edit index.js by hand after installing and add 'use strict'; at the beginning of the file then I don't get this error.

[lamweili]

Although we might not use word-wrap explicitly, it is used by dependencies such as eslint.
(eslint@>=2 <=7.0.0-alpha.3 > optionator@^0.8 > word-wrap@~1.2.3)

Editing the index.js by hand is a workaround but might be overridden by npm.

It will be good to quickly patch it back to restore compatibility instead of all dependencies changing major semver.

[bahamat]

Can confirm, I'm hitting this because of eslint.

[bahamat]

For clarification, I wasn’t suggesting editing index.js as a workaround. But if the maintainer wants to keep let, then use strict might be an alternative.

[lamweili]

@doowb, any chance of merging and releasing it soon?

[lamweili]

It seems like @doowb is busy. How about @jonschlinkert?

[lamweili]

Bump! @doowb @jonschlinkert