chore: upgrade TypeScript to v5.9 - #22861
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThe changes across the repository consist primarily of updating the TypeScript dependency from version Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (18)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
E2E results are ready! |
emrysal
left a comment
There was a problem hiding this comment.
LGTM! not a huge fan of beta modes but then again it's TS.
|
@emrysal oh actually I didn't mean this to be merged. I wanted to test how much of work is needed to upgrade to 5.9 (turned out just a little). but yeah, it's typescript. should be good to go with even beta. |
What does this PR do?
Visual Demo (For contributors especially)
A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).
Video Demo (if applicable):
Image Demo (if applicable):
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
Checklist