TGViewer
msavin | Backend & PHP msavin | Backend & PHP @msavin_php · 794 subscribers
Post #3 1.23K
Вчера написал PR в Rector (инструмент автоматического рефакторинга) 💻

Идея PR - добавить named arguments по всей кодовой базе в один клик
PR мой не приняли, так как не совпадает с планом развития самого инструмента (смотрите обсуждение), поэтому вынес в отдельный repo

Что примечательно, ранее я уже открывал 2 Issue (1, 2) в этот проект, и их тоже не приняли 🙄, зато Tomas Votruba поставил мне звездочку на один из проектов

Идея же рождалась долго: сначала Валентин Удальцов на своем хардкорном курсе обратил внимание на то, что всегда пишет имена аргументов для читаемости

После этого я раза 2 к ряду искал баг связанный с тем, что я перепутал порядок аргументов для str_contains (вместо сена писал иголку)

Затем на работе я сделал таску в другой команде, а лид команды на ревью требовал добавлять везде имена аргументов 😢
Мне было лень искать все места, куда надо добавить, но существующего правила я не нашел

Так и родилась идея написать свое правило. На все ушло часов 5. Как говорится, зачем делать за 10 минут, если можно автоматизировать за 10 часов, верно?😄
GitHub feat: add AddNamedArgumentsRector by savinmikhail · Pull Request #6678 · rectorphp/rector-src In two words this feature doing this: - (new DateTimeImmutable())->format('Y-m-d'); + (new DateTimeImmutable())->format(format: 'Y-m-d'); Also for func...
  • 👍 8
More from @msavin_php
  1. Jul 3, 2026Я писал кучу всяких библиотек и утилит, но только одна "стрельнула" и на днях пробила 100к…
  2. Jun 8, 2026Выложил видос, где разбираю чужой тех на предмет ошибок, а не прохожу собес самолично. Как…
  3. Apr 8, 2026Неочевидное поведение variadic + named arguments Вот люблю пхп и ненавижу одновременно - п…
  4. Apr 7, 2026Новое видео про распределенные транзакции, saga pattern, tcc, 2pc, eventual & strong consi…
  5. Mar 27, 2026Зарелизил наконец-то ролик про кэш в сисдизе Если упустил что-то важное, дайте знать 🙂
  6. Feb 13, 2026Есть относительно популярный вопрос на собесах - про множественное наследование в пхп Тут…
Threads Profile ViewerView any public Threads profile without an account.Open ThreadLook →Writing with AI? Make it sound human.Metric37 rewrites AI drafts so they read naturally. Free AI detector, 1,500 words free.Try Metric37 →