-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat!: bump minimum Node version to 18.12 and TypeScript to 4.8.4 #531
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #531 +/- ##
==========================================
- Coverage 80.59% 75.50% -5.10%
==========================================
Files 35 50 +15
Lines 6575 4821 -1754
Branches 440 630 +190
==========================================
- Hits 5299 3640 -1659
+ Misses 1276 1180 -96
- Partials 0 1 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
33335d5
to
d1d0755
Compare
d1d0755
to
8243e50
Compare
4f2b6a7
to
a720b0b
Compare
@JoshuaKGoldberg I updated a few things. LGTM now |
PR Checklist
status: accepting prs
Overview
Removes the following methods that rely on deprecated parts of TypeScript:
isInputFiles
(ts.SyntaxKind.InputFiles
)isUnparsedPrologue
(ts.SyntaxKind.UnparsedPrologue
)isUnparsedSourceText
(ts.isUnparsedPrepend
,ts.isUnparsedTextLike
)isUnparsedSyntheticReference
(ts.SyntaxKind. UnparsedSyntheticReference
)Also bumps all devDependencies to latest (
ncu -u
) to work on the latest Node 22 and 23 in CI.💖