Вчера написал PR в Rector (инструмент автоматического рефакторинга) 💻
Идея PR - добавить named arguments по всей кодовой базе в один клик
PR мой не приняли, так как не совпадает с планом развития самого инструмента (смотрите обсуждение), поэтому вынес в отдельный repo
Что примечательно, ранее я уже открывал 2 Issue (1, 2) в этот проект, и их тоже не приняли 🙄, зато Tomas Votruba поставил мне звездочку на один из проектов
Идея же рождалась долго: сначала Валентин Удальцов на своем хардкорном курсе обратил внимание на то, что всегда пишет имена аргументов для читаемости
После этого я раза 2 к ряду искал баг связанный с тем, что я перепутал порядок аргументов для str_contains (вместо сена писал иголку)
Затем на работе я сделал таску в другой команде, а лид команды на ревью требовал добавлять везде имена аргументов 😢
Мне было лень искать все места, куда надо добавить, но существующего правила я не нашел
Так и родилась идея написать свое правило. На все ушло часов 5. Как говорится, зачем делать за 10 минут, если можно автоматизировать за 10 часов, верно?😄
Идея 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...
- (new DateTimeImmutable())->format('Y-m-d');
+ (new DateTimeImmutable())->format(format: 'Y-m-d');
Also for func...
👍7
Welcome post
Меня зовут Михаил. Я работаю тимлидом на 2-х проектах, пишу бэкенд на PHP
Люблю опенсорс, контрибьютил к примеру в Docker, ClickHouse, JetBrains, Laravel и др. Мой аккаунт на ГитХаб
На ютуб выкладываю записи техсобесов
Топ статьи на хабре:
- CI/CD
- Тестирование
Параллельно менторю ребят по PHP
Меня зовут Михаил. Я работаю тимлидом на 2-х проектах, пишу бэкенд на PHP
Люблю опенсорс, контрибьютил к примеру в Docker, ClickHouse, JetBrains, Laravel и др. Мой аккаунт на ГитХаб
На ютуб выкладываю записи техсобесов
Топ статьи на хабре:
- CI/CD
- Тестирование
Параллельно менторю ребят по PHP
GitHub
savinmikhail - Overview
savinmikhail has 99 repositories available. Follow their code on GitHub.
🤮49👍10🗿2❤1👎1