EmbarkStudios / octobors

Rust program for automerging PRs based on a few rules
Apache License 2.0
40 stars 2 forks source link

Use octocrab fork, for logging response bodies #57

Closed davidpdrsn closed 1 year ago

davidpdrsn commented 1 year ago

Checklist

Description of Changes

We're having issues with some PRs not getting merged. The error we're seeing in the logs is just

not able to automerge: Failed to merge PR: HTTP Error: error decoding response body: expected value at line 1 column 1

Thats not a lot to go by but it seems like it fails to decode the response it gets a merge API request that fails. I'd really like to see what the response actually is.

This PR moves us to a fork of octocrab I just made that logs response bodies before attempting to deserialize them. The commit is https://github.com/EmbarkStudios/octocrab/commit/9f3485382265b570116d07dee6ae95a50b6fe765

Hopefully that should give us some more insight into what's going on.

davidpdrsn commented 1 year ago

The issue seems to have been infra related. So this is no longer needed.