hexlet-correction icon indicating copy to clipboard operation
hexlet-correction copied to clipboard

[#84] add checkstyle

Open kitdim opened this issue 1 year ago • 8 comments

kitdim avatar May 08 '24 09:05 kitdim

@kitdim Привет! Полезная штука, спасибо! Круто будет еще в мейкфайл добавить команду lint, чтобы можно было отдельно проверить стилистику

Malcom1986 avatar May 16 '24 06:05 Malcom1986

А не хотите ошибки линтера поправить?)) Плюс здорово будет в CI встроить процесс проверки линтером, чтобы при пуше в репозиторий не только тесты запускались, но и линтер. И заодно версии экшенов там обновить

Malcom1986 avatar May 17 '24 08:05 Malcom1986

@Malcom1986 привет! Почти закончил, подскажи плиз, как лучше, все правки одним коммитом отправить или на каждый файл один коммит?

kitdim avatar May 22 '24 19:05 kitdim

@Malcom1986 привет! Почти закончил, подскажи плиз, как лучше, все правки одним коммитом отправить или на каждый файл один коммит?

Привет! Правки ошибок линтера? Одним коммитом все можно отправить, это одна атомараная задача. Плюс еще стоит в проверку включить задачу checkstyleTest, чтобы он тесты тоже чекал. Сейчас не проверяет

Malcom1986 avatar May 23 '24 07:05 Malcom1986

Странно, локально ошибок нету.

kitdim avatar May 23 '24 10:05 kitdim

Всего одна ошибка осталась https://github.com/Hexlet/hexlet-correction/actions/runs/9206408938/job/25324202791?pr=263#step:6:15

Malcom1986 avatar May 24 '24 05:05 Malcom1986

Всего одна ошибка осталась https://github.com/Hexlet/hexlet-correction/actions/runs/9206408938/job/25324202791?pr=263#step:6:15

@Malcom1986 привет, как считаешь в чем может быть проблема? Когда запускаю линтер локально, то всё ок

kitdim avatar May 24 '24 21:05 kitdim

image

kitdim avatar May 24 '24 22:05 kitdim

привет, как считаешь в чем может быть проблема? Когда запускаю линтер локально, то всё ок

Привет! Походу у тебя в ветке просто нет изменений, которые произошли в репозитории в main за время работы. Обнови свою локальную ветку и запусти еще раз линтер локально

Malcom1986 avatar May 27 '24 10:05 Malcom1986

@Malcom1986 да, в точку. Всё поправил, вроде как сбилдилось

kitdim avatar May 27 '24 16:05 kitdim