Open patricklx opened 8 months ago
@NullVoxPopuli any idea how I can fix this:
Error: node_modules/@glimmer/interfaces/index.d.ts(1,2[9](https://github.com/typed-ember/glint/actions/runs/7698348329/job/20977417907?pr=696#step:5:10)): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Error: node_modules/@glimmer/interfaces/index.d.ts(3,15): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Error: node_modules/@glimmer/interfaces/index.d.ts(4,15): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Error: node_modules/@glimmer/interfaces/index.d.ts(5,15): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Error: node_modules/@glimmer/interfaces/index.d.ts(6,15): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Error: node_modules/@glimmer/interfaces/index.d.ts(7,15): error TS2834: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.
Does skipLibCheck help?
Otherwise, glimmer has more bugs we need to fix (all relative imports should always use extensions when using node16, so it shouldn't be too bad of a fix)
Does skipLibCheck help?
Otherwise, glimmer has more bugs we need to fix (all relative imports should always use extensions when using node16, so it shouldn't be too bad of a fix)
yes, it helped!
@patricklx would there be any noticeable editor behavior changes with this PR?
(or is it "just the same, but using the new parser capabilities"?)
i'm not seeing and difference
@patricklx new versions published, can you try the .1 release?
nope, some new errors-...
Error: node_modules/@types/ember__helper/index.d.ts(3,81): error TS147[9](https://github.com/typed-ember/glint/actions/runs/7739973176/job/21103924609?pr=696#step:5:10): The current file is a CommonJS module whose imports will produce 'require' calls; however, the referenced file is an ECMAScript module and cannot be imported with 'require'. Consider writing a dynamic 'import("@glimmer/manager")' call instead.
Error: node_modules/@types/ember__helper/index.d.ts(4,53): error TS1479: The current file is a CommonJS module whose imports will produce 'require' calls; however, the referenced file is an ECMAScript module and cannot be imported with 'require'. Consider writing a dynamic 'import("@glimmer/runtime")' call instead.
Error: node_modules/@glimmer/modifier/dist/commonjs/index.d.ts(1,20): error TS1479: The current file is a CommonJS module whose imports will produce 'require' calls; however, the referenced file is an ECMAScript module and cannot be imported with 'require'. Consider writing a dynamic 'import("@glimmer/runtime")' call instead.
https://github.com/microsoft/TypeScript/issues/52529 https://github.com/microsoft/TypeScript/pull/53426
looks like @ef4 knows something about this
ah yeah, we're probably blocked on ember-source needing to actually ship as a type=module
package.
@NullVoxPopuli the error is under @types folder. So we can make that type: module? https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/ember__helper/package.json
And here? https://github.com/patricklx/glimmer.js/blob/master/packages/%40glimmer/modifier/package.json
does changing all of @types/ember*
to type=module work locally? :thinking:
does changing all of
@types/ember*
to type=module work locally? 🤔
it fixes some errors,
but also needed to fix @glimmerx/modifier
and @glimmer/modifier
.
still, at the end it says
node_modules/@types/ember__helper/index.d.ts:1:24 - error TS2307: Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
Ahh, yeah, probably because all the private relative imports need extensions at that point. Hmm
I've added skipLibCheck back.
It doesn't seem we can build without it.
@patricklx FYI 0.89.0 is out with https://github.com/glimmerjs/glimmer-vm/pull/1568 The information you need should now be in ElementNode.{path,openTag,closeTag}
: https://github.com/glimmerjs/glimmer-vm/pull/1568/commits/68509ace73a77d1b36eab9c958006c9f1c98ec06#diff-21f1f3fd9aaae780472e1cf9cf079d1a05efd0fd642f6758c963dc7e969a4b30
Should also fix #706
there are now also following nodes:
example:
afterwards completions/validations can be add for attributes and element tags In #663