-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Register extra definition files depending on file #29
Comments
@JohnnyMorganz I'm not sure how much work this issue will be to implement properly, so in the meantime, would it be possible to just expose this file after the extension is installed? If we could simply add to or replace this file (in the VS Code extensions directory) post-installation, that'd be perfect for now. |
@mikejohnstn You should be able to find the file installed at Hopefully I'll get something to sort this out soon (and probably another option to disable the auto API update if you don't need it!) |
Found the file, and works perfectly for now. Thanks! |
I updated and confirmed the new definition file is working, as well as disabling the Roblox definition file. 👍 A couple issues (and maybe why you haven't closed this yet):
|
Definitely want to improve both of these aspects, the issue right now is that definition files are passed and loaded at server startup (using command line flags). We can't display diagnostics until the initialisation handshake is complete, but we load in the file before this. Loading at startup is also why a reboot is required currently. I plan on looking into deferring the definition files until later, just need to find out a way to cleanly clear the old types and reload the new ones, which is the main hurdle right now. Forcing a reload right now is just me being lazy 😅
What I would recommend for the time being if you are iterating on your definitions file is to run the standalone CLI tool:
This should give more helpful error diagnostics if the definitions file fails to load |
Ah nice, I didn't think of that. Cheers. |
It would be good to be able to register extra definitions, e.g.
testez.d.lua
.We should put this in configuration and make it so it maps globs to extra definitions
The text was updated successfully, but these errors were encountered: