Skip to content

Gracefully turn off glint when no config at all is detected#1064

Merged
NullVoxPopuli merged 2 commits intomainfrom
nvp/better-support-for-javascript-without-tsconfig
Mar 9, 2026
Merged

Gracefully turn off glint when no config at all is detected#1064
NullVoxPopuli merged 2 commits intomainfrom
nvp/better-support-for-javascript-without-tsconfig

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Contributor

@NullVoxPopuli NullVoxPopuli commented Mar 6, 2026

This is better than the error / add squiggles when no config is present behavior we have today.

image

In future PRs:

  • action to generate tsconfig
  • action to generate jsconfig
  • maaaaaybe running glint with a default jsconfig/tsconfig if one is not detected -- this will be tricky because I'll need to bundle a bunch of stuff into the vscode extension

@NullVoxPopuli NullVoxPopuli added the bug Something isn't working label Mar 6, 2026
async function copyWasmFile() {
const sourceWasm = join(
__dirname,
'../../core/node_modules/content-tag/pkg/node/content_tag_bg.wasm',
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leftover from the package rename

@evoactivity
Copy link
Copy Markdown
Contributor

Is this replacing #1052?

Makes sense if we just get it so JS experience is not full of red squigglies then improve from there.

@NullVoxPopuli
Copy link
Copy Markdown
Contributor Author

NullVoxPopuli commented Mar 8, 2026

It is not, but #1052 got too big, so I sort of wanted to do a progressive / incremental set of improvements to the JS experience when folks have Glint installed

Makes sense if we just get it so JS experience is not full of red squigglies then improve from there.

exactly

@NullVoxPopuli NullVoxPopuli merged commit cef202a into main Mar 9, 2026
4 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/better-support-for-javascript-without-tsconfig branch March 9, 2026 15:46
@github-actions github-actions bot mentioned this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants