search

looker-open-source / look-at-me-sideways

A style guide and linter for Looker's LookML data modeling language
MIT License
126 stars 37 forks source link

New LAMS error: Error [ERR_UNKNOWN_BUILTIN_MODULE]: No such built-in module: node:readline/promises #145

Closed tkovich closed 1 year ago

tkovich commented 1 year ago

Hi Fabio! I'm getting the following error:

lams --reporting=save-yes shell: /usr/bin/bash -e {0} Parsing project... Parsing done! Getting manifest and exemption info... Error [ERR_UNKNOWN_BUILTIN_MODULE]: No such built-in module: node:readline/promises at new NodeError (node:internal/errors:387:5) at Function.Module._load (node:internal/modules/cjs/loader:867:13) at Module.require (node:internal/modules/cjs/loader:1100:19) at require (node:internal/modules/cjs/helpers:108:18) at Object. (/opt/hostedtoolcache/node/16.20.1/x64/lib/node_modules/@looker/look-at-me-sideways/lib/loaders/lams-exemptions.js:2:18) at Module._compile (node:internal/modules/cjs/loader:1198:14) at Object.Module._extensions..js (node:internal/modules/cjs/loader:1252:10) at Module.load (node:internal/modules/cjs/loader:1076:32) at Function.Module._load (node:internal/modules/cjs/loader:911:12) at Module.require (node:internal/modules/cjs/loader:1100:19) at require (node:internal/modules/cjs/helpers:108:18) at module.exports (/opt/hostedtoolcache/node/16.20.1/x64/lib/node_modules/@looker/look-at-me-sideways/index.js:131:31) at async /opt/hostedtoolcache/node/16.20.1/x64/lib/node_modules/@looker/look-at-me-sideways/cli.js:28:2 { code: 'ERR_UNKNOWN_BUILTIN_MODULE' } Error: Process completed with exit code 1.

Searched your most recently merged PR and it looks like it might be related to something here: https://github.com/looker-open-source/look-at-me-sideways/pull/143/files#diff-252e9ae9caf9f8cba76b8c29fcb25c84338a43588134da9cdc6d6766b2d2d7b2R2

Any suggestions?

fabio-looker commented 1 year ago

Yes, I'll be releasing a patch later today that will prioritize compatibility with older Node versions. In the meantime, you can also use Node v17+

fabio-looker commented 1 year ago

@tkovich Should be all done now. Patch release v3.1.1 restores compatibility with Node v14+ . Sorry for the disruption and thanks for the bug report.

tkovich commented 1 year ago

Thank you thank you, that was so fast!