Уведомления
Очистить всё

Включили rubocop-rails-omakase в старом проекте: 2300 замечаний

4 Сообщения
4 Пользователи
0 Реакции
396 Просмотры
Давид Г
Сообщения: 7
Автор темы
Разобрался с блоками тимлид
Присоединился: 10 месяцев назад
[#108]

Монолит живёт с 2019 года. После январского обновления на Rails 8.1 решили заодно поставить rubocop-rails-omakase, как в новых приложениях. Первый прогон: 2300 offenses, из них 1900 автоисправляемых. Команда разделилась: одни хотят прогнать rubocop -a одним большим PR, другие против, потому что git blame превратится в кашу. Конфиг пока такой:

inherit_gem:
  rubocop-rails-omakase: rubocop.yml

inherit_from: .rubocop_todo.yml

AllCops:
  NewCops: enable
  Exclude:
    - "db/schema.rb"
    - "bin/**/*"

Как вы заходили в линтер на живом проекте?


3 ответа
kostya.sokurov
Сообщения: 73
Держит прод бэкенд-разработчик, senior
Присоединился: 4 года назад

Большой PR с автоисправлениями не страшен, если положить его хеш в .git-blame-ignore-revs и прописать git config blame.ignoreRevsFile .git-blame-ignore-revs. Тогда blame его пропускает, и большинство редакторов этот файл тоже понимают. Остальное rubocop --auto-gen-config сложит в todo, и дальше чините по одному копу, когда дойдут руки.


Ответ
chaika
Сообщения: 4
Пишет в irb техдир небольшой студии
Присоединился: 10 месяцев назад

Можно без большого PR: линтер в CI только по изменённым файлам. Кто трогает файл, тот и чистит, за полгода уходит процентов 70


Ответ
Валерий40
Сообщения: 33
Читает чужие PR фулстек-разработчик
Присоединился: 4 года назад

omakase ещё мягкий, там большая часть копов выключена. а 1900 автоисправляемых это в основном кавычки и пробелы внутри квадратных скобок


Ответ