-
Notifications
You must be signed in to change notification settings - Fork 146
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
chore: upgrade TypeScript to v5.0 #858
chore: upgrade TypeScript to v5.0 #858
Conversation
CI for Node v12 failed because TypeScript dropped support for it in v5.1 (https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-1.html#es2020-and-nodejs-1417-as-minimum-runtime-requirements). I will change TypeScript version to v5 then. |
Hey @lesha1201! Can you update this PR to remove the ESLint changes, and get the new CI config? Thanks. |
@Belco90 I will try to do this tomorrow if I will have time. The next week I'm quite busy and probably will be able to do this only on the next weekends if I can't tomorrow. |
@lesha1201 No rush! Thanks for your help. |
49b1eb3
to
103930e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #858 +/- ##
=======================================
Coverage ? 96.23%
=======================================
Files ? 44
Lines ? 2419
Branches ? 1000
=======================================
Hits ? 2328
Misses ? 91
Partials ? 0 ☔ View full report in Codecov by Sentry. |
@Belco90 Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
🎉 This PR is included in version 6.2.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Checks
Changes
tsconfig.json
based on https://www.totaltypescript.com/tsconfig-cheat-sheet and the requirements of the projectContext
Based on #856. See 4c28716...0f6fe96 for changes.