Modified all .js files + added new directory #107
Conversation
|
@Bogdan060895 у вас в репозитории очень странно файл скопировался. возможно вы drag-n-drop делали... сейчас пусть к файлу выглядит так: походу его неправильно перенесли из архива из предыдущего репозитория |
| @@ -1,3 +1,6 @@ | |||
| // const { resolve } = require('path') | |||
| // const root = resolve(__dirname, '') | |||
|
|
|||
| const { writeReactComponent, readSourceFile } = require('@root/utils'); | |||
There was a problem hiding this comment.
этот файл сейчас содержит код который у нас не выполнится. нужно переделать и вставить шаблон который есть у нас в других файлах теста
|
Должно быть как в файле del.js? первые 5 констант нам не нужны, вообще, в этом файле? |
да, пока что у нас во всех файлах пустые тесты |
|
сделаю merge этих изменений и установлю jest правильно |
|
Нужно сделать 2 файла в папке del c одинаковым содержанием? не совсем понимаю |
|
просто удалить index сейчас. оставим только пустой del.test на сейчас |
|
удалил |
|
гуд. тогда сделайте sync и потом откройте пулл реквест с изменениями |
|
почему-то идет ошибка по этому пункту LGTM analysis: JavaScript при пулл реквесте |
|
Это я подключил дополнительный анализатор. Это оч интересный сервис. Они
используют научный подход и эта хрень работает на нескольких языках без
проблем
…On Fri, Jun 3, 2022, 8:26 PM Bogdan060895 ***@***.***> wrote:
почему-то идет ошибка по этому пункту LGTM analysis: JavaScript при пулл
реквесте
—
Reply to this email directly, view it on GitHub
<#107 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALGWDV46FMNIRKLI6MGAA3VNI56FANCNFSM5XHHKL5A>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
|
Что требуется сделать в этом таске на данном этапе ? Немного запутался в следующем шаге |
|
Этот пулл реквест я принял. тут больше ничего нужно делать. а таски, которые висят на вас есть в разделе Issues |
No description provided.