Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: bump php 8.2 #99

Open
wants to merge 3 commits into
base: 2.x.x
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:

strategy:
matrix:
php: ['8.1', '8.2']
php: ['8.2', '8.3', 'nightly']

steps:
- uses: actions/checkout@v4
Expand Down
10 changes: 5 additions & 5 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,22 +14,22 @@
}
],
"require": {
"php": "^8.1 || ^8.2",
"php": "^8.2",
"ext-json": "*",
"ext-pdo": "*",
"azjezz/psl": "^2.7.0",
"doctrine/dbal": "^3.6.6",
"doctrine/dbal": "^3.7.0",
"doctrine/orm": "^2.16.2",
"guzzlehttp/guzzle": "^7.8.0",
"guzzlehttp/psr7": "^2.6.1",
"psr/http-message": "2.0"
},
"require-dev": {
"doctrine/coding-standard": "^12.0.0",
"infection/infection": "^0.27.0",
"maglnet/composer-require-checker": "^4.6.0",
"infection/infection": "^0.27.3",
"maglnet/composer-require-checker": "^4.7.1",
"malukenho/mcbumpface": "^1.2.0",
"phpunit/phpunit": "^10.3.3",
"phpunit/phpunit": "^10.3.5",
"roave/security-advisories": "dev-master",
"staabm/annotate-pull-request-from-checkstyle": "^1.8.5"
},
Expand Down
2 changes: 1 addition & 1 deletion src/Soql/BoundValuesSeparator.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/BoundValuesSeparator.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 14 and the first side effect is on line 14.

declare(strict_types=1);

Expand All @@ -11,7 +11,7 @@
use function method_exists;
use function sprintf;

final class BoundValuesSeparator
final readonly class BoundValuesSeparator
{
private function __construct()
{
Expand Down
2 changes: 1 addition & 1 deletion src/Soql/DBAL/Result.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

use function Psl\invariant;

class Result extends DBALResult
final class Result extends DBALResult
{
private DriverResult $storedResult;

Expand Down
2 changes: 1 addition & 1 deletion src/Soql/Driver/Result.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/Driver/Result.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 12 and the first side effect is on line 12.

declare(strict_types=1);

Expand All @@ -9,7 +9,7 @@
use PDO;
use PDOException;

final class Result implements DriverResultInterface
final readonly class Result implements DriverResultInterface
{
public function __construct(private SoqlStatement $statement)
{
Expand Down
4 changes: 2 additions & 2 deletions src/Soql/Factory/Http/RequestThrottler.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/Factory/Http/RequestThrottler.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 12 and the first side effect is on line 12.

declare(strict_types=1);

Expand All @@ -9,11 +9,11 @@
use function explode;
use function trim;

final class RequestThrottler
final readonly class RequestThrottler
{
public const HEADER = 'Sforce-Limit-Info';

public function __construct(public readonly int $used, public readonly int $total)
public function __construct(public int $used, public int $total)
{
if ($this->used >= $this->total) {
throw RequestThrottlingException::create($this->used, $this->total);
Expand Down
8 changes: 4 additions & 4 deletions src/Soql/Factory/HttpAuthorizedClientFactory.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/Factory/HttpAuthorizedClientFactory.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 12 and the first side effect is on line 12.

declare(strict_types=1);

Expand All @@ -9,12 +9,12 @@

use function sprintf;

final class HttpAuthorizedClientFactory implements AuthorizedClientFactory
final readonly class HttpAuthorizedClientFactory implements AuthorizedClientFactory
{
public function __construct(
private readonly AccessTokenFactory $accessTokenFactory,
private readonly string $salesforceInstance,
private readonly string $apiVersion,
private AccessTokenFactory $accessTokenFactory,
private string $salesforceInstance,
private string $apiVersion,
) {
}

Expand Down
2 changes: 1 addition & 1 deletion src/Soql/Payload.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/Payload.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 15 and the first side effect is on line 15.

declare(strict_types=1);

Expand All @@ -12,7 +12,7 @@

use const JSON_THROW_ON_ERROR;

final class Payload
final readonly class Payload
{
/** @param mixed[] $values */
public function __construct(
Expand Down
2 changes: 1 addition & 1 deletion src/Soql/SoqlDriver.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<?php

Check warning on line 1 in src/Soql/SoqlDriver.php

View workflow job for this annotation

GitHub Actions / CI (8.2)

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 23 and the first side effect is on line 23.

declare(strict_types=1);

Expand All @@ -20,7 +20,7 @@
use function array_merge_recursive;
use function Psl\invariant;

class SoqlDriver implements Driver, ExceptionConverter
readonly class SoqlDriver implements Driver, ExceptionConverter
{
/** {@inheritDoc} */
public function connect(array $params): Connection
Expand Down
Loading