Skip to content

Commit

Permalink
Fix CI issues checking out old commits (#467)
Browse files Browse the repository at this point in the history
The used `github.event.pull_request.merge_commit_sha` seems to be broken
and has the old commit always. Instead we try now use the
`github.event.pull_request.head.sha`. This is required because we use
secrets and need so use `pull_request_target` instead of `pull_request`
event which behaves differently.
  • Loading branch information
alexander-schranz authored Dec 27, 2024
1 parent 4c8fcfb commit e4514d3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Console/ReindexCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@ public function __invoke(
EngineRegistry $engineRegistry,
): int {
/** @var \DateTimeImmutable|null $dateTimeBoundary */
$dateTimeBoundary = $this->datetimeBoundary ? new \DateTimeImmutable((string) $this->datetimeBoundary) : null;
$dateTimeBoundary = $this->datetimeBoundary ? new \DateTimeImmutable((string) $this->datetimeBoundary) : null; // @phpstan-ignore-line
/** @var array<string> $identifiers */
$identifiers = \array_filter(\explode(',', (string) $this->identifiers));
$identifiers = \array_filter(\explode(',', (string) $this->identifiers)); // @phpstan-ignore-line

$reindexConfig = ReindexConfig::create()
->withIndex($this->indexName)
Expand Down

0 comments on commit e4514d3

Please sign in to comment.