Когда перестанут игнорировать Pull-request`s

Вопрос к разработчикам webAsyst.

Подскажите кто ответсвенный за данный раздел https://github.com/webasyst/shop-script/pulls готовые пул реквесты висят годами и не отказа, ни принятия ни просто комментария.

Разработчики тратят свое время чтобы оформить pull-request а его просто игнорируют.

Неужели сложно специалисту просмотреть и хотя бы отписаться по каждому pull-request или хотя бы скажите что webAsyst не просматривает и не использует этот раздел и что создавать pull-request-ы не нужно

9 ответов

  • 10

    Уважаемые разработчики webAsyst мы видим некоторые положительные тенденции, в частности модерации плагинов, ответы на вопросы и т.д.

    Но вопрос с небольшими но нужными внедрениями в ваш фремеворк стоит довольно остро.

    Понимаем что скорее всего у вас не хватает возможностей и квалифицированных специалистов. Но у вас под боком почти 200 разработчиков, многие из которых уже знают каждый винтик в этом фремеворке.

    Вполне вероятно некоторые задачи можно переложить на такой своеобразный оутсорс. Условия всегда можно обговорить.

  • 3
    Eugen Nichikov 18 сентября 2017 12:47 #

    уии!!!

    https://github.com/webasyst/webasyst-framework/pull/223

  • 3
    Eugen Nichikov 21 июля 2017 16:44 #

    Поддерживаю

    Кроме того есть ещё вот этот раздел

    https://github.com/webasyst/webasyst-framework/pulls

    и вот этот раздел

    https://github.com/webasyst/hub/pulls

    Раньше как-то более активно работа с PR велась.

    • +1
      Eugen Nichikov Eugen Nichikov 21 июля 2017 16:51 #

      А ещё типа "PR" на форуме, например

      https://support.webasyst.ru/14690/kontakty-pro-nekorrektno-formiruetsya-ssylka-otkryt-v-kontaktakh/

      уже несколько обновлений не вносят 2 строчки, которые ошибку исправляют.

  • 3
    Eugen Nichikov 31 июля 2017 20:19 #

    И всё-равно нет ответа.

    • +3
      Павел Трофимов Павел Трофимов 31 июля 2017 20:48 #

      - Webasyyyyyyyyyyyyyyyyst........
      - Мля... мля... мля..... - привычно откликнулось эхо.... :)

    • +1
      Алексей | shop-script.org Алексей | shop-script.org 2 августа 2017 23:21 #

      Вот это больше всего удивляет, что нет ответа.
      Ну написали бы уже куда идти, все бы знали, что отказано.
      А так и не да и не нет...

  • 2
    Марк 24 июля 2017 16:01 #

    Поддерживаю. Давно предлагаю ввести бонусы за подобную работу сообщества.

  • 2
    Гена Рыжов 4 августа 2017 13:59 #

    Насколько я понял, перед тем как делать PR лучше сначала завести тему на форуме и обсудить, а просто так кидать возможно абсолютно не нужные коды нет смысла.

    • +2
      Eugen Nichikov Eugen Nichikov 4 августа 2017 14:14 #

      Что мешает обсуждать готовый код?

      Кроме того, даже обсуждение на форуме и публикация PR со ссылками на темы не помогает.


      А вот "Принятая" тема на форуме. Без PR, т.к. нет репы для Контактов ПРО. Но 2 строчки уже почти год не вносятся.


      P.S. Раньше это хоть как-то работало... приходил Музыченко раз в пару месяцев и на почту сыпались уведомления о принятых/отклонённых PR.

    • +2
      J. B. diGriz J. B. diGriz 4 августа 2017 14:33 #

      Скажем так, в описании PR было бы неплохо детально объяснять что это и зачем нужно. И быть уверенным, что это не сломает совместимость.

      Крайне нежелательно (ну просто пздц, как нежелательно, можно считать нельзя) менять дескрипторы конструкторов и методов системных классов. Например добавлять, пусть и необязательные, параметры к методам. Нет никаких гарантий, что класс не отнаследовал в своем плагине/приложении какой-нибудь разработчик и такое изменение способно обрушить работу его продукта. WA, если сильно припирает, в этих случаях использует func_get_args, кстати.

  • 1
    Axelvaisper 2 августа 2017 23:17 #

    Как вариант в подобных ситуациях большие организации выделяют community edition сборки

  • 1
    Minya Jam 7 сентября 2017 21:44 #

    Я тоже зареквестил, но до сих пор не внесли правки, а оно там достаточно критичное по новому функционалу.

  • 1
    fedoric 9 сентября 2017 01:01 #

    Не хотелось бы показаться "язвой" но уважаемые разработчики из webasyst обратите внимание на https://github.com/webasyst/webasyst-framework/iss...

    А то Вы мне по поводу экранирования всю плеш проели аж глаз дергается, а у самих вон оно че.

    • +1
      Гена Рыжов Гена Рыжов 19 сентября 2017 20:58 #

      Зря ты так хаки раздаешь!

    • +2
      Михаил Ушенин Михаил Ушенин Webasyst 20 сентября 2017 09:49 #

      Это не "язва", а просто сообщение об ошибке. Каждое такое сообщение оформляйте в виде отдельной темы с пометкой "Сообщение об ошибке". Все такие сообщения передаются разработчикам для исправления. За сообщения об ошибках вам большое спасибо!

Добавить ответ

Чтобы добавить комментарий, зарегистрируйтесь или войдите