-
Notifications
You must be signed in to change notification settings - Fork 185
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
Bump jscodeshift to fix ast error with TS satisfies operator #6986
Conversation
size-limit report 📦
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
e2e tests |
👀 Docs deployed
Commit f452bb8 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6986 +/- ##
=======================================
Coverage 83.41% 83.41%
=======================================
Files 352 352
Lines 10538 10538
Branches 3498 3498
=======================================
Hits 8790 8790
Misses 1748 1748
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e5003a9
to
c7ba8aa
Compare
4a4d48d
to
f452bb8
Compare
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.
Класс, @BlackySoul, спасибо, что обновила README по поводу появления лишних круглих скобочек.
Если б мог, то поставил бы 👍
Описание
Старая версия
jscodeshift
не могла переваритьTS
-файлы с новым синтаксисом, поэтому смирились с некоторыми нежелательными моментами + ап версии позволит порешать security alertИзменения
Сделали ап версии
jscodeshift
, обновили снепшоты + порешали старуюTODO
(вместо удаления файла просто выбросили ненужные части кода)