You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
One of the large benefits of moving to JS configs instead of JSON is intelliSense, but launchpad is not currently equipped to take advantage of that. The JS Language service looks for typescript declaration files (.d.ts) for intellisense in most IDEs, but because launchpad is written in JS, no declaration files are being generated.
What we can do is add a tsconfig.json file to each of the launchpad package directories, and configure them with the following options:
Then add a tsc step to the changeset publish action. This will run the ts type-checker on our js + jsdoc, generating declaration files and publishing them alongside the js to npm.
This also has the added benefit of more thorough type checking across the board.
The text was updated successfully, but these errors were encountered:
claytercek
changed the title
Stricter typing / IntelliSense
Stricter type checking / IntelliSense
Aug 23, 2023
Love this idea! I was wondering a few times what it would take to generate these. Might also help with private vs public functions down the road, right?
Seems like a pretty low-hanging fruit feature to try out as. Marking it as medium high priority.
Mentioned in our plugin API discussion (#114)
One of the large benefits of moving to JS configs instead of JSON is intelliSense, but launchpad is not currently equipped to take advantage of that. The JS Language service looks for typescript declaration files (
.d.ts
) for intellisense in most IDEs, but because launchpad is written in JS, no declaration files are being generated.What we can do is add a
tsconfig.json
file to each of the launchpad package directories, and configure them with the following options:Then add a
tsc
step to the changeset publish action. This will run the ts type-checker on our js + jsdoc, generating declaration files and publishing them alongside the js to npm.This also has the added benefit of more thorough type checking across the board.
The text was updated successfully, but these errors were encountered: