Skip to content
Snippets Groups Projects

search: more consistently report timeout errors instead of "No results" when `timeout:` directive is present

Created by: ijt

This is progress on #3294 .

This also includes a helpful hint for a longer timeout directive in the error message.

Test plan: Manually tested with queries like func timeout:1ms on a sourcegraph instance with a single repo

Before: Screen Shot 2019-04-12 at 4 55 25 PM

After: Screen Shot 2019-04-12 at 4 42 37 PM

The commits shown on github are screwed up and include things from a previous PR. Sorry about that.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: ijt

    I don't know how to fix this mess with the commits, so I'm starting a new PR with some cherry-picks. Closing this PR out.

Please register or sign in to reply
Loading