Mark Hemmings

Results 11 comments of Mark Hemmings

Same thing for method not allowed. There should be a way to define a custom error response format for all internal errors and any response from rest.Error

Yeah milliseconds make much more sense. I was thinking maybe to allow this without it being a breaking change, the duration unit could be on the Logger? e.g. ```go type...

@bradleyfalzon / @wlynch Let me know what you think :)

I've just noticed the `Token()` method only returns a string, so there is currently no way to get the expires date out to "save" an access token. Not sure the...

Any updates on this? What's left to do, and is there anything I can help with?

I have found it to return an error on `Ping()` too, but this only happens when `SetMaxIdleConns(0)` (which is perfectly [valid](https://golang.org/pkg/database/sql/#DB.SetMaxIdleConns) way to remove pooling). The error goes away when...

The PAT I'm using has these permissions. Definitely excessive!

Hi @williammartin, he's the debug output: ``` gh pr merge --auto "$PR_URL" ⣾* Request at 2023-11-29 17:43:25.875218 +0000 GMT m=+0.069728085 * Request to https://api.github.com/graphql > POST /graphql HTTP/1.1 > Host:...

Seems to be calling the incorrect API? Should be calling `enqueuePullRequest`?

@williammartin Permissions are the same as yours (see screenshot below) and apply to the repo being merged. I'm fairly confident the PAT was used correctly, but I will do a...