Привел в порядок описание API плагина БазовыеУтверждения#842
Conversation
|
@sulfur17 Исправления полезные.
Только вот с этим вопрос. |
|
@sulfur17 может быть, сделать просто 2 примера
|
|
@artbear ваш метод очень интересный, мне даже в голову не приходило что так можно. Но если хотите знать мое мнение - писать лишние пустые параметры гораздо проще чем разобраться какой параметр в таком плавающем подходе за что будет отвечать. Вот допустим какие параметры я тут передаю? Я конечно исправлю как скажете, но прошу вас обдумать мои аргументы. |
|
@ Я согласен, что подобное поведение может и путать. такое поведение уже много лет ) сейчас исправить уже сложно, т.к. куча клиентов использует именно такой интерфейс и не хотелось бы им ломать их тесты. предлагаю оставить как есть, только добавить простые примеры вызова в 2х вариантах, как я предложил выше. если не хочешь, тогда я готов замержить и в текущем варианте ) |
|
у них ничего не сломается, потому что я в коде ничего не менял. Я просто дал другие имена параметрам в документации. |
сделанные изменения:
@vanessa-opensource/Collaborators - просьба прокомментировать и проверить