Adding workflow tests and fix installation in documentation

[wellwelwel]

Hi @aashutoshrathi, I would like to contribute by automating existing tests for every new submitted PR and for pushes to master branch.

Having millions of downloads, it's good to always make sure nothing is broken with new changes (except when intentionally).

---

About the package-lock.json (2,246 additions by performing npm i), it's necessary to install the correct packages and perform the CI tests.

npm ci

---

Also, I noticed two errors in README.md:

word-wrap/README.md

Line 10 in 8498142

$ npm install --save word-wrap

word-wrap/README.md

Line 16 in 8498142

var wrap = require('word-wrap');

• Then, I just fixed it to your npm package.

---

Feel free to ask anything 🚀

[wellwelwel]

Just complementing, for now, it wasn't possible to test versions earlier than NodeJS 14, because Mocha requires the NodeJS 14.0.0 or higher.

[wellwelwel]

Another point I haven't touched on, is trimEnd:

word-wrap/index.js

Line 10 in 8498142

const trimmedLines = lines.map((line) => line.trimEnd());

---

@aashutoshrathi, I don't think a polyfill is necessary, just set the engine in package.json to >=10.3.0.
But I leave it to you decide.

[aashutoshrathi]

Another point I haven't touched on, is trimEnd:

word-wrap/index.js

Line 10 in 8498142

const trimmedLines = lines.map((line) => line.trimEnd());

@aashutoshrathi, I don't think a polyfill is necessary, just set the engine in package.json to >=10.3.0. But I leave it to you decide.

But I want to be sure that if we do the polyfill, will we be vulnerability free? As I think the polyfill uses regex

[aashutoshrathi]

So, it is okay to publish without package-lock.json, but we need that for CI it, right?

[aashutoshrathi]

Hey! @wellwelwel Can you point this PR to v1.2.7 and take a pull of the same branch once.

[aashutoshrathi]

Also, can you add the CI Badge too?

[wellwelwel]

So, it is okay to publish without package-lock.json, but we need that for CI it, right?

I'm away from PC now, but the package-lock.json probably will be ignored because the files field into package.json.

[wellwelwel]

Hey! @wellwelwel Can you point this PR to v1.2.7 and take a pull of the same branch once.

Alright, but after I wake up 🙋🏻‍♂️

[wellwelwel]

Closing in favor of:

• Adding workflow tests and fix installation in documentation #8