Skip to content

Commit

Permalink
Merge pull request #15 from portier/feat/compat
Browse files Browse the repository at this point in the history
Broaden supported ranges of php and deps
  • Loading branch information
stephank authored Nov 29, 2023
2 parents 6a21df0 + 8cfef90 commit d7e9362
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 13 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,9 @@ jobs:
name: Check
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
php-versions: ['8.1', '8.2', '8.3']
php-versions: ['7.4', '8.0', '8.1', '8.2', '8.3']
steps:

- name: Checkout
Expand Down
10 changes: 5 additions & 5 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@
}
},
"require": {
"fgrosse/phpasn1": "^2.2",
"lcobucci/clock": "^3.0.0",
"lcobucci/jwt": "^5.2.0",
"guzzlehttp/guzzle": "^7.3"
"fgrosse/phpasn1": "^2.3.1",
"lcobucci/clock": "^2.0.0 || ^3.0.0",
"lcobucci/jwt": "^4.1.0 || ^5.0.0",
"guzzlehttp/guzzle": "^7.2.0"
},
"require-dev": {
"phpunit/phpunit": "^10.4.2",
"phpunit/phpunit": "^9.5 || ^10.4.2",
"phpstan/phpstan": "^1.2.0",
"friendsofphp/php-cs-fixer": "^3.14"
}
Expand Down
18 changes: 11 additions & 7 deletions src/VerifyResult.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,17 @@
namespace Portier\Client;

/** The result of a call to `Client::verify`. */
class VerifyResult
final class VerifyResult
{
public function __construct(
/** The verified email address. */
public string $email,
/** State that was carry over from the call to `authenticate`. */
public ?string $state = null,
) {
/** The verified email address. */
public string $email;
/** State that was carry over from the call to `authenticate`. */
public ?string $state;

/** @internal */
public function __construct(string $email, ?string $state)
{
$this->email = $email;
$this->state = $state;
}
}

0 comments on commit d7e9362

Please sign in to comment.