-
Notifications
You must be signed in to change notification settings - Fork 124
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
Feature request - Support libraries #113
Comments
For anyone who ends up here, I managed to use it in my libraries creating a /// <reference types="@total-typescript/ts-reset" /> I guess it will depend on your bundling configuration, but in mine it works properly and it doesn't include any reference to What do you think @mattpocock? Is it a good idea? If you don't see anything wrong with it I can open a PR to add this on the README. And to be honest, if it works maybe it's even better to just do this all the time, that way you don't have to give different instructions whether you're working on a library or an app :). |
I have a slightly different use case I'm wondering if it's possible to be supported or if we could get some pointers on: I thought about using Note that I wouldn't do that in a runtime library. This is solely to help coworkers use TS Reset without having to forget to configure it properly. Or that it even exists. |
Instead of not supporting libraries we can find a clever way to only conditionally apply ts-reset. Then libraries can explicitly enable and typecheck but otherwise it's disabled by default
The text was updated successfully, but these errors were encountered: