GHC 2017-08-12

21 comments.

, https://git.io/v7H4m in jarun/googler
I'm only doing word completion at the moment, because I didn't consider that case. (I'm accustomed to putting queries in a single argument). Unfortunately, completing multiple arguments as if they should be concatenated into a single argument is very tricky.

, https://git.io/v7H4Y in jarun/googler
Another bug. Damn...

, https://git.io/v7H4O in jarun/googler
@jarun Directly pushed this fix since further testing of the bash version (which was undertested) surfaced a bug.

, https://git.io/v7H43 in jarun/googler
Autocomplete queries
====================

Using the https://www.google.com/complete/search API. See `completer_fetch_completions` in `googler` for details.

Functionality added to completion scripts for all three shells. No opt-out at the moment.

, https://git.io/v7H4s in zmwangx/googler
Okay, I took a look and realized I don't need to relearn anything to finish the bash version, so I just implemented it.

, https://git.io/v7HCx in zmwangx/googler
I'll get to that. Maybe tonight if I have some spare time. Or maybe a few days later; after all I have a 15-hour international flight back to China tomorrow.

, https://git.io/v7HCp in jarun/googler
Oh yeah, I had to refresh the page constantly too. I do have email notifications on, though.

, https://git.io/v7HCh in Homebrew/homebrew-core
imagemagick@6 6.9.9-7
=====================

Created with `brew bump-formula-pr`.

---

Please remember to
```
brew mirror imagemagick@6
```
when merging.

<img src="https://i.imgur.com/s8TMOSU.png" width="40" height="53">

, https://git.io/v7HCj in Homebrew/homebrew-core
imagemagick 7.0.6-7
===================

Created with `brew bump-formula-pr`.

---

Please remember to
```
brew mirror imagemagick
```
when merging.

<img src="https://i.imgur.com/s8TMOSU.png" width="40" height="53">

, https://git.io/v7HWe in jarun/googler
@jarun Absolutely no offense, but are you able to see my comments in the thread https://github.com/zmwangx/googler/commit/e3f52f19b77303fd01b9e45690b990c6e5fc4903#commitcomment-23617093? Judging from your comments I feel like my comments are somehow hidden from you...

, https://git.io/v7HWv in zmwangx/googler
Weird, you're responding as if you can't see my comments on this thread...

, https://git.io/v7HWf in jarun/googler
Include Python version in debug output
======================================

Why ask for it when we can extract that ourselves.

, https://git.io/v7HWJ in zmwangx/googler
Yeah, that's exactly what c617e3a (current tip of the branch) does.

, https://git.io/v7HWU in zmwangx/googler
Are you using c617e3a? I can't reproduce on 3.5 or earlier.

, https://git.io/v7HWT in zmwangx/googler
Should be fixed in c617e3a.

, https://git.io/v7HWk in zmwangx/googler
Oh, okay, loading from bytes is new in 3.6. https://docs.python.org/3/library/json.html#json.loads I'll push out a patch in a second.

, https://git.io/v7HWI in zmwangx/googler
Huh what... Your Python version is?

, https://git.io/v7HWL in zmwangx/googler
Or just `curl 'https://www.google.com/complete/search?client=psy-ab&q=goo'`.

, https://git.io/v7HWt in zmwangx/googler
I suppose if you run `googler --complete goo` it also doesn't work?

, https://git.io/v7HZM in zmwangx/googler
That's right.

, https://git.io/v7HZD in zmwangx/googler
Any suggestion for a non-`string` equivalent of `[[ $arg != -* ]]` in fish? Even this `string` version is pretty barbaric, but that was what I could come up with after browsing fish's docs for five minutes. I only tested on fish 2.6.0, by the way.