Skip to content

Instantly share code, notes, and snippets.

View anton-kotik's full-sized avatar

Anton Kotik anton-kotik

View GitHub Profile
@anton-kotik
anton-kotik / checklist.md
Created February 16, 2017 05:57
Code Review Checklist
  • В коде есть миграции? Показать Саше. Тяжелые ALTER TABLE запускать только ночью в выходные, перед запуском такого альтера надо сделать deploy-lock, убедиться через processlist что альтер начал работать, потом убрать deploy-lock.
  • Экранируем всё в html шаблонах.
  • [ ]
@anton-kotik
anton-kotik / README.md
Last active September 3, 2019 13:44
dv-post-example
  • Удаляем лишние теги и пустые атрибуты:
    <\/?(span|div|strong|br)\s*\/?>|<p>&nbsp;</p>|<\w+> </\w+>|\s+(\w+)\="" (разделитель страницы)
  • Теги:
    Feature &nbsp; -> <span class="dv-feature">Feature</span>&nbsp;
    Improvement &nbsp; -> <span class="dv-improvement">Improvement</span>&nbsp;
    Bug &nbsp; -> <span class="dv-bug">Bug</span>&nbsp;
  • Удаляем лишние переносы строк: (\r?\n){3,} -> \n\n
  • Ссылки на задачи в YouTrack:
    <a href="[^"]+">\s*(\w+\-\d+)\s*</a> -> <a href="https://youtrack.dv.team/issue/$1" class="dv-task" target="_blank">$1</a>
  • Прочие ссылки: