dep(npm): bump @cordova/[email protected] w/ lint corrections

[breautek]

Platforms affected

Web/JS

Motivation and Context

Upgrades our lint config to the latest available version and applied --fix

Depends on: #93

Description

ESLint was being dumb in one location:

Where lint was confident that id was never reassigned even though it was just after the win function is declared. It wanted the id declare to be a const, which obviously failed cause it needed to be initialised with a value at declaration.

There was 2 workarounds, bring the let id into the describe block slope or move the declaration down and use const. I opted for the latter since there was no other tests, but I have no objection to bring the declaration to the describe block either.

Testing

Ran npm run lint

Checklist

• I've run the tests to see all new and existing tests pass
• I added automated test coverage as appropriate for this change
• Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
• If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
• I've updated the documentation if necessary