shubham-j-sde commented on PR #287: URL: https://github.com/apache/kvrocks-controller/pull/287#issuecomment-2742464936
I see that, but 1. wouldn't now rather be best time to include typescript since website is just in beginning development, rather to include after having bunch of functions, exports, dependencies and what not. changing dependencies and including type check is a pain, trust me. 2. we are using Next.js so typescript support comes [built-in](https://nextjs.org/docs/pages/getting-started/installation#set-up-typescript) 3. Next.js [community encourages](https://www.reddit.com/r/nextjs/comments/14xyzpx/using_next_without_typescript/?rdt=47445) using Typescript; following community standards helps in resolving bugs and including new features much more simple and easy. 4. and as @SharonIV0x86 you very well highlighted, Typescript is more safe and modern practice. Honestly, i believe current code lacks even some Next.js module standards, not to mention basic web accessibility standards. and that's what we need to work upon. your thoughts? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
