Skip to content

Commit

Permalink
refactor(HasSoarBinary): rename getDefaultSoarBinary to defaultSoarBi…
Browse files Browse the repository at this point in the history
…nary

- Renamed the method getDefaultSoarBinary to defaultSoarBinary for consistency and clarity
  • Loading branch information
guanguans committed Apr 23, 2024
1 parent d27174a commit 6e66c0f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/Concerns/HasSoarBinary.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ protected function getEscapedSoarBinary(): string
return escape_argument($this->soarBinary);
}

protected function getDefaultSoarBinary(): string
protected function defaultSoarBinary(): string
{
if (OS::isWindows()) {
return __DIR__.'/../../bin/soar.windows-amd64'; // @codeCoverageIgnore
Expand Down
2 changes: 1 addition & 1 deletion src/Soar.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class Soar implements Contracts\Soar
public function __construct(array $options = [], ?string $soarBinary = null)
{
$this->setOptions($options);
$this->setSoarBinary($soarBinary ?? $this->getDefaultSoarBinary());
$this->setSoarBinary($soarBinary ?? $this->defaultSoarBinary());
}

public static function create(array $options = [], ?string $soarBinary = null): self
Expand Down

0 comments on commit 6e66c0f

Please sign in to comment.