Skip to content

Commit

Permalink
fix is_finished state handling
Browse files Browse the repository at this point in the history
  • Loading branch information
thatportugueseguy committed Dec 19, 2024
1 parent a8ba00f commit 35dcf91
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/state.ml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ let set_repo_pipeline_status { state } (n : Github_t.status_notification) =
let build_number = Util.Build.get_build_number_exn ~build_url in
let is_finished =
match is_pipeline_step, n.state with
| true, (Success | Failure | Error) -> true
| false, (Success | Failure | Error) -> true
| _ -> false
in
let finished_at =
Expand Down
20 changes: 20 additions & 0 deletions state.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"repos": {
"https://github.com/thatportugueseguy/buildkite-starter": {
"pipeline_statuses": { "buildkite-tests": { "main": {} } },
"pipeline_commits": {
"buildkite-tests": {
"s1": [
"047410c599ffe20d7c36c17d0de4ce019f0bf72f",
"189fe7c1ef516602c0fe250f6ef291d2e27bfab6",
"1c6fe59e1ec03c5b4d3eaf0c6e51506a7938ca14",
"21f461c00bc5aae193546c8e264bcffefa83c863",
"9623221a2edac7c2c3135e64d7fedf25cc8be0f9",
"eb06fc937d0ffb965f3f5810bd321f352502f1ed"
],
"s2": []
}
}
}
}
}

0 comments on commit 35dcf91

Please sign in to comment.