http://test-results.appspot.com/flakiness search should use ":" |
|||
Issue descriptionThe search box uses: Team "team-name" It should instead be consistent with crbug and other apps and do: team:team-name directory:path/to/dir
,
Dec 5 2016
Thanks for reporting this. Currently we do not allow combining the search patterns, i.e. you can either search for "team:Blink" or "directory:foo/bar", but not "team:Blink directory:foo/bar". The latter would be non-trivial to implement. Do you think we should do this?
,
Dec 11 2016
I was just suggesting to change the syntax to match by using colon instead of space+quotes. Totally fine to make multiple search terms a separate bug and mark that as p3.
,
Dec 12 2016
Ack. Filed issue 673298.
,
Dec 12 2016
,
Dec 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/d0c5612792c762c45aecbae4cf48c9e5a17e2532 commit d0c5612792c762c45aecbae4cf48c9e5a17e2532 Author: Sergiy Byelozyorov <sergiyb@chromium.org> Date: Mon Dec 12 18:45:14 2016 Use lower-case prefix and colons to match Code Search style BUG= 670971 Change-Id: I6f4d7fce9e977e35e70f612a5a5debf20ca5413b Reviewed-on: https://chromium-review.googlesource.com/418784 Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> [modify] https://crrev.com/d0c5612792c762c45aecbae4cf48c9e5a17e2532/go/src/infra/appengine/test-results/frontend/src/test-results-flakiness.html
,
Dec 13 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by estaab@chromium.org
, Dec 5 2016