Ian Schneider

> However, I think you might actually be saying that some dependency is using gulplog, which makes it available in the global scope but you aren't using the new gulp-cli,...

> Is the solution for gulp-util.log to check (the yet-to-be-created) hasGulpCli, instead of hasGulplog? It could be? Unfortunately, I've moved on and no longer use gulp in my day-to-day work.

There's a number of `geometry: Optional[dict]` though the typing intention is now including `str` (ref) I haven't dug enough to see if it's even possible (or useful) to try to...

In discussion, structlog was considered too opinionated for a library - e.g if a downstream user would benefit from structured logging, then they can introduce that and integrate with stdlib...

Consider using a click ParamType for reuse

In discussion, we decided that there are 2 actionable aspects to this ticket: * reduce reliance of jq in CLI docs (especially for introductions) but recognize in the spirit of...

So far, so good! Tested out and found things to be pretty straight-forward and it's definitely nice to have a means to grab/refresh a token My initial review: **Good:** -...

Nice work! I'm definitely a big :+1: on the single Planet class. I did a small test listing my searches and then concurrently executing each search using both sync and...

Another approach - by extending DataClient and overriding every function (as sync and with altered type hints), we get the best of both (can inherit doc strings and get correct...

looks good so far! one thing we've been trying to do w/ tests is combine the sync/async calls into a single case since the setup and assertions are the same,...