-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
Remove deprecated root-level options #397
Comments
Looks like an interesting idea, but why not create a luaconfig.json instead then? |
I don't think it matters much if its in its own object or just in the config root. Might be slightly nicer to have them grouped. But such a change would break all existing projects, which kinda sucks. Im absolutely against having another config file, that will just make things more messy and harder to maintain imo. |
Can't it be the same as with old compiler directives? Deprecate old names, use them as a fallback and remove in 1.0.0 (#365).
+1 on this, most projects already have quite a number of config files. |
I think indeed a path forward would be to support both for a while and eventually get rid of the global settings. I also agree another setting file might just end up cluttering the root of the project (look at the stuff in this project alone). |
This removes the need to add
lua
before option names and would look better if more options would be added.Another module that extends tsconfig, awesome-typescript-loader, is doing the same.
The text was updated successfully, but these errors were encountered: