Форки официальных репозиториев всё ещё помечены как официальные

Клиент сделал форк discourse-rss-polling, и я установил его. :tada:

Но похоже, что у меня всё ещё установлена официальная версия:

image

Оставлю как упражнение проверить, что указанный коммит отсутствует в репозитории. При наведении курсора на плагин я вижу, что он ссылается на форкнутый репозиторий.

5 лайков

Хм, что скажешь об этом, @sam?

1 лайк

Это, безусловно, баг. Думаю, простое решение — убедиться, что плагин всегда загружается с discourse.org на GitHub, чтобы он был помечен как официальный.

Технически, однако, всё ещё можно довольно легко обмануть.

Проверка того, что конкретный SHA соответствует нашему SHA, была бы единственным 100% точным способом, но для этого нам пришлось бы запустить ещё один сервис.

На мой взгляд, я бы отнёс это к категории «что-то, что нужно исправить в течение следующих 2–3 лет».

8 лайков

Меня тоже недавно это подвело. Галочка определенно меня несколько времени сбивала с толку, и я провел кучу ненужного тестирования, прежде чем заметил репозиторий в app.yml.

Простое исправление, похоже, вполне достаточно.

Как и в случае со слишком короткими сообщениями, суть не в том, чтобы сделать невозможным обман теста, а лишь в том, чтобы случайно не обмануть его. :slight_smile:

1 лайк

Самое простое решение здесь специфично для плагина Docker Manager. Я не уверен, насколько правильно запускать дополнительную команду git или искать удалённые репозитории git для каждого плагина при каждой загрузке системы.

Но плагин Docker Manager мог бы взять на себя эту дополнительную работу и добавлять большой красный значок к официальным плагинам, которые, по-видимому, являются форками.

Я добавлю это :arrow_double_up: в следующий релиз, так что это произойдёт в течение следующих 6 месяцев или около того.

pr-welcome, если кто-то хочет поэкспериментировать с Docker Manager.

6 лайков

Я добавил индикатор форка в этот пул-реквест:

6 лайков

Эта тема была автоматически закрыта через 4 дня. Новые ответы больше не принимаются.