Heath Stewart

Results 419 comments of Heath Stewart

I'll rename the switch and rejigger a bit of the logic and let you know. Probably tonight. Looks like I need to rebase again, but at least this has been...

> Me, reading the jq docs. I frequently have to switch tools - and always have to refer back to docs - that use either `jq` syntax, `JSONPath`, or `JMESPath`....

@williammartin this should be ready now. I removed the "experimental" notes given this is an additional flag that still requires `--paginate`, but feel free, of course, to update the text...

Wait: something is wrong. Tests pass, but I just tried it in a use case and it still emits separate pages and a final blank `{}`. This worked previously. 😕...

@williammartin I realized the problem: I forgot, to maintain backcompat, this only works when piping or redirecting stdout. When I was testing it, I was just printing to the terminal....

PS: seems the `pr-auto` bot doesn't like it when a PR is readied again: says the PR is already linked to a project. It's not required, but just wanted to...

@LangLangBart the merging is shallow, so the first array found is merged. Even if we completely rewrote `mergo`, how would we know how deep to go? Should we merge the...

> My solution to this --paginate issue (in Powershell at least) was to do this: > > ``` > $jsonResults = gh api -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" --paginate...

> I would have used a repo query for this organization as an example, but any example would work if > it involves redirecting to stdout or piping to another...

@andyfeller @williammartin I believe this is backward compatible now through some manual testing, and restored / cleaned up tests. You might look at `api_test.go` anew instead of a diff from...