rust-lang / triagebot

Automation/tooling for Rust spaces
https://triage.rust-lang.org
Apache License 2.0
169 stars 75 forks source link

Log more information about why commits_in_range isn't working. #1811

Closed ehuss closed 1 month ago

ehuss commented 1 month ago

This is a followup to https://github.com/rust-lang/triagebot/pull/1801 to further investigate why it is not working correctly. From what I can tell, the /commits endpoint seems to be returning results out of order. I'm expecting them to be in commit order, but that doesn't seem to be the case (but I still can't reproduce locally). This adds a log message to help me confirm that theory. If that is the case, I will likely need to add something that traverses the parent commits to get the correct set of commits, or perhaps consider using the GraphQL endpoint which might be more reliable.