-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Request: TypeScript support (~C# parity) #334
Comments
Noted - in the meantime would this be helpful? https://github.com/microsoft/node-api-dotnet |
@jsegeren , parity would be great for C#. If you want to help with that effort we would love to get PRs from you in this branch - https://github.com/microsoft/semantic-kernel/tree/experimental-typescript |
+1 on this feature, really interesting! I'll try to participate in the typescript branch |
+1 |
@evchaki : +1, I would also love to contribute for typescript, Can you please help us in getting started, roadmap, task to contribute. |
### Motivation and Context <!-- Thank you for your contribution to the chat-copilot repo! Please help reviewers and future users, providing the following information: 1. Why is this change required? 2. What problem does it solve? 3. What scenario does it contribute to? 4. If it fixes an open issue, please link to the issue here. --> Incorrect controller route/method signature causing build error ### Description <!-- Describe your changes, the overall approach, the underlying design. These notes will help understanding how your code works. Thanks! --> ``` D:\a\chat-copilot\chat-copilot\webapi\Controllers\MaintenanceController.cs(50,55): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [D:\a\chat-copilot\chat-copilot\webapi\CopilotChatWebApi.csproj] ``` ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [x] The code builds clean without any errors or warnings - [x] The PR follows the [Contribution Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [x] All unit tests pass, and I have added new tests where possible - [x] I didn't break anyone 😄
Closing since we currently do not have plans to add a TypeScript version of the SDK. This may change in the future. |
I hope the future is now :). Typescript would be a great language for semantic kernel since you can iterate much faster. I really think python is inferior to Typescript as a programming language. |
Any updates on TypeScript support @matthewbolanos ? If we are even supporting Java, there is no reason, why (the Microsoft owned) TypeScript has no support as the most used programming language. |
Hi - we have a project using Next.JS and would love to leverage SK on the backend.
A full TypeScript port (ideally 1:1 parity with C#) would be truly valuable and appreciated.
As part of this, a NPM distribution of the TS SDK (i.e. https://registry.npmjs.org).
Happy to discuss/explain the scenarios more.
Thanks!
The text was updated successfully, but these errors were encountered: