Участник сообщества предложил жирный PR, добавляющий UPSERT в Cycle.
Я сейчас сфокусирован на других задачах, поэтому нужна помощь с этой.
Можно ревьювить чисто в своих компетенциях, главное обозначить их в комментарии.
Но желательно знать, что такое UPSERT и как он используется.
Что можно ревьювить:
- API в Query Builder'е.
- Тесты (корректность и полнота тестов).
- Комментарии.
- Кишочки.
На картинке показано, что нажимать, чтобы предлагать изменения: выделяете несколько строк, жмякаете
+
, кнопка +-
для предложения правок.Please open Telegram to view this post
VIEW IN TELEGRAM
👍11
Forwarded from Alexander Makarov
Кто хочет помочь Yii3, но не уверен в своём коде, есть две лёгких, но очень полезных задачи:
- https://github.com/yiisoft/friendly-exception/issues/37
- https://github.com/yiisoft/error-handler/issues/148
Нужно наделать скриншотов, показывающих фичи, и закинуть в README пакетов + оформить в виде pull request.
- https://github.com/yiisoft/friendly-exception/issues/37
- https://github.com/yiisoft/error-handler/issues/148
Нужно наделать скриншотов, показывающих фичи, и закинуть в README пакетов + оформить в виде pull request.
GitHub
Add screenshot to readme · Issue #37 · yiisoft/friendly-exception
Need a nice screenshot from a friendly exception from Yii3 added to README.
👍7
Плагин для IDEA Ultimate / PHPStorm с языковой поддержкой синтаксиса PHP опкодов.
Если у вас есть опкоды и вы не знаете куда их деть, то можно положить в файл с расширением .phpo и плагин сделает подсветку и небольшую навигацию.
На текущий момент сделано достаточно для комфортного чтения опкодов, но в будущем появится и инспекция с анализом ваших опкодов.
Где и как их собирать удобно – это тема для другого плагина. Он уже в паблике, но полноценной интеграции с Opcodes Language нет.
Please open Telegram to view this post
VIEW IN TELEGRAM
JetBrains Marketplace
PHP Opcodes Language - IntelliJ IDEs Plugin | Marketplace
GitHub | Telegram | Donation PHP Opcodes Language support Features: syntax highlightingblock foldingvariable referencesopcodes documentation (soon)rules inspections...
👍4
Dmitrii
#php
Для любителей изучать новые вещи, могу предложить заняться добавлением примеров опкодов для доки при наведении на опкод:
- Открываете сборник опкодов
- Клонируете определение опкода в плагине
- Описываете его назначение и примеры: пхп + опкоды
Готовые опкоды можно взять отсюда, остальные придется искать / изучать исходники
👩💻 Ссылка на файл, в который нужно добавлять
👩💻 Ссылка на php-src со списком последних опкодов
Для любителей изучать новые вещи, могу предложить заняться добавлением примеров опкодов для доки при наведении на опкод:
- Открываете сборник опкодов
- Клонируете определение опкода в плагине
- Описываете его назначение и примеры: пхп + опкоды
Готовые опкоды можно взять отсюда, остальные придется искать / изучать исходники
Please open Telegram to view this post
VIEW IN TELEGRAM
🔥4
Forwarded from Mikhail Savin
Rector-правило для соблюдения AAA паттерна в тестах
По результатам моего опроса почти половина разработчиков пользуются этим паттерном при написании тестов, но я лично периодически забываю "отбивать" секции комментами
Теперь можно делегировать это Rector'у - добавил правило EnforceAaaPatternRector.
Оно просто вставляет комментарии (и удаляет старые, если они неверно расположены), например:
Если же ваши тесты изначально не структурированы под это, то будут мисплейсы. Например, если вы делаете запрос, ассерты на него, потом новый запрос и новые ассерты (так часто тестят валидацию/авторизацию).
Надеюсь, правило поможет сделать такие нарушения более явными на кодревью
По результатам моего опроса почти половина разработчиков пользуются этим паттерном при написании тестов, но я лично периодически забываю "отбивать" секции комментами
Теперь можно делегировать это Rector'у - добавил правило EnforceAaaPatternRector.
Оно просто вставляет комментарии (и удаляет старые, если они неверно расположены), например:
public function testFoo(): void
{
// Arrange
$date = new DateTimeImmutable('2025-01-01');
// Act
$formatted = $date->format('Y-m-d');
// Assert
self::assertEquals('2025-01-01', $formatted);
}
Если же ваши тесты изначально не структурированы под это, то будут мисплейсы. Например, если вы делаете запрос, ассерты на него, потом новый запрос и новые ассерты (так часто тестят валидацию/авторизацию).
Надеюсь, правило поможет сделать такие нарушения более явными на кодревью
Telegram
msavin
Используете AAA (arrange-act-assert/given-when-then) при тестировании?
да / нет / не пишу тесты
да / нет / не пишу тесты
👍2🤔2👎1