Comments 20
Сучки и задоринки, за которые мы зацепились
Зацепился за сучек неработающей ссылки на демку, в урл закралась "."
Взял сучкарез и отрезал её, пролез)
Видимо сломалось.
Oops. Something went wrong
We can't find the GitHub repository you're looking for.
Что вводите?
И полный URL, и только имя_пользователя/репа (кстати, ожидаемый формат никак не описан), а даже просто в URL писал -- все одинаково.
Только что попробовал ввести https://github.com/manticoresoftware/manticoresearch-go - всё сработало. Вот так делал:
https://github.manticoresearch.com/tafia/quick-xml не работает. Пробовал и https://github.com/tafia/quick-xml и tafia/quick-xml.
Ваш пример работает.
Да, вижу, обсужу с разработчиками. Спасибо.
Пофиксили
А почему влитые запросы на слияния у вас считаются «открытыми»? Кажется, что это нелогично. Вот что выдаёт поиск «Везде» (Everywhere) в состоянии «открыто» (State = Open):
А переходим в сам запрос на слияние — а он уже влит более недели назад:
Что интересно, если искать среди конкретно запросов на слияние (Pull Requests), то этот запрос находится только среди «закрытых»:
Потому что тут вы не фильтруете по pull requests (search in "Everywhere") и то, что выделено - это комментарий, а не pull request. Текст pull request'а для этого комментария приведён просто для справки. Но раз это интуитивно непонятно, то видимо нужно доработать UI.
Даже если так, у комментария нет никаких статусов открыт / закрыт, следовательно, это статусы объекта, к которому относится комментарий. И мне кажется неправильным показывать комментарии к тем объектам, которые мы явно исключили из поиска фильтром, то есть, комментариев к закрытым запросам на слияние в результатах поиска быть не должно, если мы ищем только среди «открытых».
Либо вообще исключать комментарии из результатов поиска, если стоит какой-то статус (а у комментария статусов нет) и добавить новый фильтр «без статуса» для поиска среди комментариев. Но мне кажется, первый вариант интуитивно понятнее и проще будет.
Мы решили, что нужно распространять фильтр Open / Closed на комментарии тоже. У самих комментариев такого статуса нет, но у родительских объектов он есть, вот его и будем использовать. Делать будем через новую фичу в мантикоре - JOIN. Это будет отличной демонстрацией этой функциональности.
Ну тогда этот фильтр банально работает сейчас неправильно :)
Да. С нашей с вами точки зрения неправильно. С чьей-то (кто считает, что если фильтр неприменим к чему-то, то он должен просто игнорироваться, а усложнять UI не стоит) правильно. В любом случае это open source проект: если кто-то сделает PR до того, как мы сделаем то, что задумали, то мы с радостью его примем. Спасибо, что подсветили эту проблему.
Спасибо. Заработало
Пофиксили.
Демо: Поиск на GitHub с помощью Manticore Search