Skip to content

Commit

Permalink
v3.0 Release (#70)
Browse files Browse the repository at this point in the history
v3.0 supports Doctrine DBAL 4+ and PHP 8.1+
  • Loading branch information
asanikovich authored Sep 26, 2024
1 parent f7d0dda commit d2927af
Show file tree
Hide file tree
Showing 6 changed files with 26 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:

strategy:
matrix:
php-versions: ['8.0', '8.1', '8.2']
php-versions: ['8.1', '8.2', '8.3']
fail-fast: false

services:
Expand Down
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/.idea/
/vendor/
/.phpunit.result.cache
/.phpunit.cache/
/composer.lock
/phpunit.xml
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ Doctrine DBAL driver for ClickHouse - an open-source column-oriented database ma

**Driver is suitable for Symfony or any other framework using Doctrine.**

* [v3](https://github.com/FriendsOfDoctrine/dbal-clickhouse/releases?q=v3.) supports Doctrine DBAL 4+ and PHP 8.1+
* [v2](https://github.com/FriendsOfDoctrine/dbal-clickhouse/releases?q=v2.) supports Doctrine DBAL 3+ and PHP 8.0+
* [v1](https://github.com/FriendsOfDoctrine/dbal-clickhouse/releases?q=v1.) supports Doctrine DBAL 2+ and PHP 7.1+

Expand Down
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,15 @@
}
],
"require": {
"php": "^8.0",
"php": "^8.1",
"ext-pcre": "*",
"ext-mbstring": "*",
"doctrine/dbal": "^4.0",
"smi2/phpclickhouse": "^1.0"
},
"require-dev": {
"doctrine/coding-standard": "^4.0 || ^9.0",
"phpunit/phpunit": "^9.5"
"phpunit/phpunit": "^9.5 || ^10.0 || ^11.0"
},
"autoload": {
"psr-4": {
Expand Down
21 changes: 19 additions & 2 deletions phpunit.xml.dist
Original file line number Diff line number Diff line change
@@ -1,10 +1,27 @@
<?xml version="1.0" encoding="utf-8"?>
<phpunit bootstrap="vendor/autoload.php" colors="true">
<?xml version="1.0" encoding="UTF-8"?>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.0/phpunit.xsd"
bootstrap="vendor/autoload.php"
cacheResultFile=".phpunit.cache/test-results"
beStrictAboutOutputDuringTests="true"
beStrictAboutTodoAnnotatedTests="true"
failOnRisky="true"
failOnWarning="true">
<testsuites>
<testsuite name="ClickHouse">
<directory>tests</directory>
</testsuite>
</testsuites>

<source>
<include>
<directory suffix=".php">src</directory>
</include>
</source>

<coverage cacheDirectory=".phpunit.cache/code-coverage">
</coverage>

<php>
<const name="phpunit_ch_host" value="localhost"/>
<const name="phpunit_ch_port" value="8123"/>
Expand Down
4 changes: 2 additions & 2 deletions src/ClickHouseSchemaManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ public function listTableIndexes(string $table): array
$tableView = $this->_getPortableViewDefinition(['name' => $table]);

preg_match(
'/MergeTree\(([\w+, \(\)]+)(?= \(((?:[^()]|\((?2)\))+)\),)/mi',
'/MergeTree\(([\w+, ()]+)(?= \(((?:[^()]|\((?2)\))+)\),)/mi',
$tableView->getSql(),
$matches
);
Expand All @@ -80,7 +80,7 @@ public function listTableIndexes(string $table): array
);

return [
new Index(
'primary' => new Index(
current(array_reverse(explode('.', $table))) . '__pk',
$indexColumns,
false,
Expand Down

0 comments on commit d2927af

Please sign in to comment.