-
Notifications
You must be signed in to change notification settings - Fork 270
[compiler] Move capitalize to compiler #7199
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
Conversation
All changed packages have been documented.
|
You can try these changes here
|
I wonder, should we consider a separate export path for name utils? Could also include case conversions, which is also VERY commonly needed. |
talked with @bterlson and while exporting more right now (like rexporting import {captialize} from "@typespec/compiler/casing"; |
Makes sense, and just to be clear this means we want to add a new entry to package.json for this subpath export correct? Happy to do so, just wanted to make sure I understand 👍 |
yep |
Co-authored-by: Timothee Guerin <[email protected]>
Follow-up from #7153 (comment) - we noticed that many packages have their own copy of this helper so this PR moves it to the compiler where packages can use it.