Skip to content

Commit

Permalink
Merge pull request #21 from Nekland/feature/php-82-update
Browse files Browse the repository at this point in the history
  • Loading branch information
Nek- authored Mar 6, 2023
2 parents 6bdde50 + 19cf697 commit 3500537
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,12 @@ jobs:
strategy:
fail-fast: false
matrix:
php: [ 7.4, 8.0 ]
php: [ 7.4, 8.0, 8.1 ]
os: [ ubuntu-latest ]
phpunit-versions: ['latest']
include:
- os: [ ubuntu-latest ]
php: 8.1
php: 8.2
composer-flag: "--ignore-platform-reqs"

- os: [ ubuntu-latest ]
Expand Down
9 changes: 4 additions & 5 deletions src/Utils/Tempfile/TemporaryDirectory.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,29 +15,28 @@

class TemporaryDirectory
{
/** @var bool */
private $wasAlreadyExisting;

/** @var string */
private $dir;

/** @var bool */
private $removed;

/** @var string[] */
private $files;

/**
* @param null|string $dir
* @param string $prefix
*/
public function __construct(string $dir = null, string $prefix = 'phpgenerated')
{
$this->files = [];
$this->removed = false;
$this->wasAlreadyExisting = false;
if (null !== $dir && \is_dir($dir)) {
if (!is_writable($dir)) {
throw new LogicException(\sprintf('The directory "%s" is not writeable.', $dir));
}

$this->wasAlreadyExisting = true;
$this->dir = $dir;
return;
}
Expand Down

0 comments on commit 3500537

Please sign in to comment.