Телеграмм чат группы yii3ru страница 2448

Как вы чаще всего указываете пакеты при запуске dev-tool

Анонимный опрос

13%

Не указываю, выполняю команду по всем 150 пакетам

19%

Указываю все что перечислены в packages.local.php или некоторые из них

2%

Указываю те что не упомянуты в packages.local.php

67%

Не пользуюсь dev-tool

Проголосовало: 48

Нет варианта "указываю те, с которыми работаю"

15:18пожаловаться #1

NO

Переслано от Nex Otaku

Те над которыми сейчас работаешь ты в любом случае будешь указывать, вопрос в том над каким множеством ты работаешь чаще всего:
1. Все пакеты
2. Несколько пакетов упомянутых в конфиге
3. Несколько пакетов не упомянутых в конфиге

15:18пожаловаться #2

AM

Alexander Borisov

А dev-tool автоматически синхронизирует твой форк с апстримом?

Нет.

15:19пожаловаться #3

AM

Какая практика по Code Review? Принимает PR только @samdark или любой может? Можно без Code Review заливать что-то?

Лучше подождать хотя-бы одного аппрува. Ну и тесты чтобы прошли.

15:20пожаловаться #4

AM

Сергей Предводителев

ревьювит любой, вливает в подавляющем большинстве случаев самдарк

А хочу чтобы сами :)

15:20пожаловаться #5

AM

Переслано от Nex Otaku

Те над которыми сейчас работаешь ты в любом случае будешь указывать, вопрос в том над каким множеством ты работаешь чаще всего:
1. Все пакеты
2. Несколько пакетов упомянутых в конфиге
3. Несколько пакетов не упомянутых в конфиге

2

15:21пожаловаться #6

Д

Интересно, а лейбл и ревьюверов нельзя автоматом через CI сделать?

Тогда какой смысл в этом лейбле? 🙂

15:24пожаловаться #7

А

Алексей R in Yii Framework 3

Тогда какой смысл в этом лейбле? 🙂

Норм штука, когда бот вешает нужные теги и группы. Мне понравилось как у rector это делается

15:57пожаловаться #8

NO

Тогда какой смысл в этом лейбле? 🙂

Надо спросить у того кто его придумал.

15:57пожаловаться #9

NO

Я вот никакого смысла в нём не вижу )

15:58пожаловаться #10

Д

Контрибьютер не может повесить тег и ревьюверов, это делает ревьювер.
Тег вешается тогда, когда работа по ПР закончена. Созданный ПР не означает завершения работ. Тег вешает ревьювер, когда понимает, что ПР нужно ревьювить. Добавляется команда ревьюверов тогда, когда ПР может создать дискуссии, да и вообще для того, чтобы все увидели изменения.

Как это делать автоматически?

16:00пожаловаться #11

AB

Alexander Borisov in Yii Framework 3

Контрибьютер не может повесить тег и ревьюверов, это делает ревьювер.
Тег вешается тогда, когда работа по ПР закончена. Созданный ПР не означает завершения работ. Тег вешает ревьювер, когда понимает, что ПР нужно ревьювить. Добавляется команда ревьюверов тогда, когда ПР может создать дискуссии, да и вообще для того, чтобы все увидели изменения.

Как это делать автоматически?

по фразе needs review от автора ПР

16:01пожаловаться #12

Д

И об этом написать в доке и не давать создавать ПР, пока контрибьютер не согласится с лицензионном соглашением? 🙂

16:04пожаловаться #13

AB

Alexander Borisov in Yii Framework 3

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

16:06пожаловаться #14

Д

Плохо ответственность перекидывать на контрибьютеров.

16:07пожаловаться #15

NO

@predvoditelev не ставится yiisoft-app.

Ругается на пакеты "yiispft/i18n-*", таких пакетов при этом нет в демо ) Там какой-то "yiisoft/translator", видимо репозиторий переехал, что-то склеили? Надо поправить "app" а то он даже не устанавливается.

16:08пожаловаться #16

NO

image_2021-03-01_16-08-41.png

(63.57 Кб)

16:08пожаловаться #17

AB

Alexander Borisov in Yii Framework 3

Плохо ответственность перекидывать на контрибьютеров.

если не контрибьютеры то кто? мэйнтенеры?

16:09пожаловаться #18

NO

Контрибьютер не может повесить тег и ревьюверов, это делает ревьювер.
Тег вешается тогда, когда работа по ПР закончена. Созданный ПР не означает завершения работ. Тег вешает ревьювер, когда понимает, что ПР нужно ревьювить. Добавляется команда ревьюверов тогда, когда ПР может создать дискуссии, да и вообще для того, чтобы все увидели изменения.

Как это делать автоматически?

Мне только что выше написали что надо самому вешать этот лейбл. Похоже в команде нет единого мнения на этот счёт?

16:10пожаловаться #19

Д

Мне только что выше написали что надо самому вешать этот лейбл. Похоже в команде нет единого мнения на этот счёт?

Попробуй сделать. Раньше этого нельзя было делать.