dojo / cli

:rocket: Dojo - command line tooling.
http://dojo.io
Other
26 stars 34 forks source link

Bump lint-staged to fix observable error #288

Closed JamesLMilner closed 5 years ago

JamesLMilner commented 5 years ago

Type: bug

Description:

When trying to commit, the pre commit hook was failing with the follow error:

husky > npm run -s precommit (node v10.16.1)

/home/james/Code/SitePen/dojo/cli/node_modules/any-observable/register.js:29
                throw new Error('Cannot find any-observable implementation nor' +
                ^

Error: Cannot find any-observable implementation nor global.Observable. You must install polyfill or call require("any-observable/register") with your preferred implementation, e.g. require("any-observable/register")('rxjs') on application load prior to any require("any-observable").
    at loadImplementation (/home/james/Code/SitePen/dojo/cli/node_modules/any-observable/register.js:29:9)
    at register (/home/james/Code/SitePen/dojo/cli/node_modules/any-observable/loader.js:32:18)
    at Object.<anonymous> (/home/james/Code/SitePen/dojo/cli/node_modules/any-observable/index.js:2:39)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (internal/modules/cjs/helpers.js:25:18)

This PR brings lint-staged up to date and fixes the issue (as per https://github.com/okonet/lint-staged/issues/416#issuecomment-458830064)