sopel-irc / sopel-github

GitHub plugin for Sopel
Other
3 stars 13 forks source link

github: simplify getting PR merge status #123

Closed dgw closed 1 year ago

dgw commented 1 year ago

merged_at is null if the PR is either open, or closed unmerged.

Not 100% certain that field is guaranteed to be populated for every PR on GitHub, regardless of age, but I think it's acceptable to get rid of the second API request in exchange for a small risk of giving slightly incorrect output in a very small number of cases.