svelte-adapter-firebase icon indicating copy to clipboard operation
svelte-adapter-firebase copied to clipboard

Consider using `eslint-plugin-jsdoc`

Open jthegedus opened this issue 4 years ago • 1 comments

I am sure the quality of the JSDoc could be improved. We should investigate using this eslint plugin to help:

  • https://github.com/gajus/eslint-plugin-jsdoc

jthegedus avatar Sep 16 '21 11:09 jthegedus

Got this error :shrug:

svelte-adapter-firebase on  main [$!] is 📦 v0.12.0 via ⬢ v14.17.5 took 3s 
➜ pnpm add -D eslint-plugin-jsdoc
 WARN  [email protected] requires a peer of eslint@^6.0.0 || ^7.0.0 but none was installed.
 WARN  @sveltejs/kit > @sveltejs/vite-plugin-svelte: [email protected] requires a peer of svelte@>=3.19.0 but none was installed.
 WARN  @sveltejs/kit: @sveltejs/[email protected] requires a peer of svelte@^3.34.0 but none was installed.
 WARN  @sveltejs/[email protected] requires a peer of svelte@^3.39.0 but none was installed.
 WARN  [email protected] requires a peer of firebase-admin@^8.0.0 || ^9.0.0 but none was installed.
 WARN  1 other warnings
Packages: +5
+++++
Progress: resolved 198, reused 193, downloaded 5, added 5, done

devDependencies:
+ eslint-plugin-jsdoc 36.1.0
svelte-adapter-firebase on  main [$!] is 📦 v0.12.0 via ⬢ v14.17.5 took 4s 
➜ pnpm run test

> [email protected] test /home/jthegedus/projects/svelte-adapter-firebase
> xo && uvu tests/unit

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/[email protected]/node_modules/array-union/index.js
require() of ES modules is not supported.
require() of /home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/[email protected]/node_modules/array-union/index.js from /home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/@[email protected][email protected]/node_modules/@typescript-eslint/typescript-estree/node_modules/globby/index.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename /home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/[email protected]/node_modules/array-union/index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/[email protected]/node_modules/array-union/package.json.

    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1089:13)
    at Module.load (internal/modules/cjs/loader.js:937:32)
    at Function.Module._load (internal/modules/cjs/loader.js:778:12)
    at Module.require (internal/modules/cjs/loader.js:961:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at Object.<anonymous> (/home/jthegedus/projects/svelte-adapter-firebase/node_modules/.pnpm/@[email protected][email protected]/node_modules/@typescript-eslint/typescript-estree/node_modules/globby/index.js:3:20)
    at Module._compile (internal/modules/cjs/loader.js:1072:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
    at Module.load (internal/modules/cjs/loader.js:937:32)
    at Function.Module._load (internal/modules/cjs/loader.js:778:12)
 ERROR  Test failed. See above for more details.

jthegedus avatar Sep 16 '21 11:09 jthegedus