diff options
| author | Francois Kritzinger <francois@codesynthesis.com> | 2024-12-05 08:15:27 +0200 |
|---|---|---|
| committer | Francois Kritzinger <francois@codesynthesis.com> | 2024-12-06 08:26:28 +0200 |
| commit | 0b58c632f857b35c816e078e858bbbf6df2a4774 (patch) | |
| tree | f5b7c44bc7f36552d6ba5097407ec6947f2833ed /mod/mod-ci-github.cxx | |
| parent | 1f453b6c38d928fb40ffd288702ef23d1944daf9 (diff) | |
Post-review changes
Diffstat (limited to 'mod/mod-ci-github.cxx')
| -rw-r--r-- | mod/mod-ci-github.cxx | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/mod/mod-ci-github.cxx b/mod/mod-ci-github.cxx index fed8013..ba2dece 100644 --- a/mod/mod-ci-github.cxx +++ b/mod/mod-ci-github.cxx @@ -764,9 +764,13 @@ namespace brep return true; } - string gh_conclusion (gh_to_conclusion (*conclusion, warning_success)); + // Note that the case mismatch is due to GraphQL (gh_conclusion()) + // requiring uppercase conclusion values while the received webhook values + // are lower case. + // + string gh_conclusion (gh_to_conclusion (conclusion, warning_success)); - if (*cs.check_suite.conclusion != gh_conclusion) + if (icasecmp (*cs.check_suite.conclusion, gh_conclusion) != 0) { error << sub << ": conclusion " << *cs.check_suite.conclusion << " does not match service data conclusion " << gh_conclusion; |