Когда перестанут игнорировать Pull-request`s
Вопрос к разработчикам webAsyst.
Подскажите кто ответсвенный за данный раздел https://github.com/webasyst/shop-script/pulls готовые пул реквесты висят годами и не отказа, ни принятия ни просто комментария.
Разработчики тратят свое время чтобы оформить pull-request а его просто игнорируют.
Неужели сложно специалисту просмотреть и хотя бы отписаться по каждому pull-request или хотя бы скажите что webAsyst не просматривает и не использует этот раздел и что создавать pull-request-ы не нужно
9 ответов
Уважаемые разработчики webAsyst мы видим некоторые положительные тенденции, в частности модерации плагинов, ответы на вопросы и т.д.
Но вопрос с небольшими но нужными внедрениями в ваш фремеворк стоит довольно остро.
Понимаем что скорее всего у вас не хватает возможностей и квалифицированных специалистов. Но у вас под боком почти 200 разработчиков, многие из которых уже знают каждый винтик в этом фремеворке.
Вполне вероятно некоторые задачи можно переложить на такой своеобразный оутсорс. Условия всегда можно обговорить.
уии!!!
https://github.com/webasyst/webasyst-framework/pull/223
Поддерживаю
Кроме того есть ещё вот этот раздел
https://github.com/webasyst/webasyst-framework/pulls
и вот этот раздел
https://github.com/webasyst/hub/pulls
Раньше как-то более активно работа с PR велась.
А ещё типа "PR" на форуме, например
https://support.webasyst.ru/14690/kontakty-pro-nekorrektno-formiruetsya-ssylka-otkryt-v-kontaktakh/
уже несколько обновлений не вносят 2 строчки, которые ошибку исправляют.
И всё-равно нет ответа.
- Webasyyyyyyyyyyyyyyyyst........
- Мля... мля... мля..... - привычно откликнулось эхо.... :)
Вот это больше всего удивляет, что нет ответа.
Ну написали бы уже куда идти, все бы знали, что отказано.
А так и не да и не нет...
Поддерживаю. Давно предлагаю ввести бонусы за подобную работу сообщества.
ачивки!!!11! :)
Насколько я понял, перед тем как делать PR лучше сначала завести тему на форуме и обсудить, а просто так кидать возможно абсолютно не нужные коды нет смысла.
Что мешает обсуждать готовый код?
Кроме того, даже обсуждение на форуме и публикация PR со ссылками на темы не помогает.
А вот "Принятая" тема на форуме. Без PR, т.к. нет репы для Контактов ПРО. Но 2 строчки уже почти год не вносятся.
P.S. Раньше это хоть как-то работало... приходил Музыченко раз в пару месяцев и на почту сыпались уведомления о принятых/отклонённых PR.
Скажем так, в описании PR было бы неплохо детально объяснять что это и зачем нужно. И быть уверенным, что это не сломает совместимость.
Крайне нежелательно (ну просто пздц, как нежелательно, можно считать нельзя) менять дескрипторы конструкторов и методов системных классов. Например добавлять, пусть и необязательные, параметры к методам. Нет никаких гарантий, что класс не отнаследовал в своем плагине/приложении какой-нибудь разработчик и такое изменение способно обрушить работу его продукта. WA, если сильно припирает, в этих случаях использует func_get_args, кстати.
Есть хорошо описанные PR и плохо описанные PR разницы никакой.
Как вариант в подобных ситуациях большие организации выделяют community edition сборки
Думаю, что у Вебасиста такого не будет.
Я тоже зареквестил, но до сих пор не внесли правки, а оно там достаточно критичное по новому функционалу.
Не хотелось бы показаться "язвой" но уважаемые разработчики из webasyst обратите внимание на https://github.com/webasyst/webasyst-framework/iss...
А то Вы мне по поводу экранирования всю плеш проели аж глаз дергается, а у самих вон оно че.
Зря ты так хаки раздаешь!
Это не "язва", а просто сообщение об ошибке. Каждое такое сообщение оформляйте в виде отдельной темы с пометкой "Сообщение об ошибке". Все такие сообщения передаются разработчикам для исправления. За сообщения об ошибках вам большое спасибо!