Skip to content

Commit

Permalink
ci: fix job names
Browse files Browse the repository at this point in the history
  • Loading branch information
m6w6 committed Jun 2, 2023
1 parent 712cc13 commit e04151d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,7 @@ jobs:
make -f scripts/ci/Makefile test
cur-dbg-zts-1:
name: php-8.2-debug-zts
name: php-8.2-nodebug-zts
env:
PHP: "8.2"
enable_debug: "no"
Expand Down Expand Up @@ -315,7 +315,7 @@ jobs:
make -f scripts/ci/Makefile test
cur-dbg-zts-2:
name: php-8.2-debug-zts
name: php-8.2-debug-nozts
env:
PHP: "8.2"
enable_debug: "yes"
Expand Down Expand Up @@ -343,7 +343,7 @@ jobs:
make -f scripts/ci/Makefile test
cur-dbg-zts-3:
name: php-8.2-debug-zts
name: php-8.2-nodebug-nozts
env:
PHP: "8.2"
enable_debug: "no"
Expand Down
10 changes: 5 additions & 5 deletions scripts/gen_github_workflow_ci.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ function yesno(array $env, string $key) : string {
} else {
$check = $key;
}
if ($env["enable_$check"] ?? null === "yes") {
if (($env["enable_$check"] ?? null) === "yes") {
return $key;
}
return "no$key";
Expand All @@ -32,14 +32,14 @@ function jobname(string $id, array $env) : string {
"enable_debug" => "yes",
"enable_maintainer_zts" => "yes",
"enable_session" => "yes",
],
],
"master" => [
// master
"PHP" => ["master"],
"enable_debug" => "yes",
"enable_zts" => "yes",
"enable_session" => "yes",
],
],
"cur-matrix" => [
// most useful for all additional versions except current
"PHP" => ["8.0", "8.1"],
Expand All @@ -51,14 +51,14 @@ function jobname(string $id, array $env) : string {
// everything disabled for current
"PHP" => $cur,
"enable_session" => "no",
],
],
"cur-dbg-zts" => [
// everything enabled for current, switching debug/zts
"PHP" => $cur,
"enable_debug",
"enable_zts",
"enable_session" => "yes",
],
],
"cur-cov" => [
// once everything enabled for current, with coverage
"CFLAGS" => "-O0 -g --coverage",
Expand Down

0 comments on commit e04151d

Please sign in to comment.