[PR #94] [MERGED] RiakStorage upgrade #114

New Issue

admin · 2026-01-23T11:35:15+01:00

admin commented

2026-01-23 11:35:15 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/KeyValueStore/pull/94
Author: @EmanueleMinotto
Created: 9/7/2019
Status: ✅ Merged
Merged: 9/20/2019
Merged by: @EmanueleMinotto

Base: master ← Head: issues/91

📝 Commits (2)

186c8e0 Add riak to Travis
8f07e47 Replace riak client with php-riak/riak-client

📊 Changes

6 files changed (+176 additions, -190 deletions)

View changed files

📝 .travis.yml (+16 -0)
📝 composer.json (+2 -3)
📝 docs/reference/configuration.rst (+5 -3)
📝 lib/Doctrine/KeyValueStore/Storage/RiakStorage.php (+51 -33)
📝 phpunit.xml.dist (+1 -0)
📝 tests/Doctrine/Tests/KeyValueStore/Storage/RiakStorageTest.php (+101 -151)

📄 Description

Fixes https://github.com/doctrine/KeyValueStore/issues/91

Things done:

drop PHP 5.5 support: riak package, Travis, PHP itself, etc... nobody is supporting it anymore
replaced riak/riak-client:dev-master with php-riak/riak-client:v1.0.0-alpha7: the first doesn't support PHP 7.0+ and isn't maintained
changed unit test for Riak with an integration test

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/KeyValueStore/pull/94 **Author:** [@EmanueleMinotto](https://github.com/EmanueleMinotto) **Created:** 9/7/2019 **Status:** ✅ Merged **Merged:** 9/20/2019 **Merged by:** [@EmanueleMinotto](https://github.com/EmanueleMinotto) **Base:** `master` ← **Head:** `issues/91` --- ### 📝 Commits (2) - [`186c8e0`](https://github.com/doctrine/KeyValueStore/commit/186c8e04d2070e858e75db556940aee8cb9aa891) Add riak to Travis - [`8f07e47`](https://github.com/doctrine/KeyValueStore/commit/8f07e47320dcaa9ecb6b6e3ec370b9295071f70f) Replace riak client with php-riak/riak-client ### 📊 Changes **6 files changed** (+176 additions, -190 deletions) <details> <summary>View changed files</summary> 📝 `.travis.yml` (+16 -0) 📝 `composer.json` (+2 -3) 📝 `docs/reference/configuration.rst` (+5 -3) 📝 `lib/Doctrine/KeyValueStore/Storage/RiakStorage.php` (+51 -33) 📝 `phpunit.xml.dist` (+1 -0) 📝 `tests/Doctrine/Tests/KeyValueStore/Storage/RiakStorageTest.php` (+101 -151) </details> ### 📄 Description Fixes https://github.com/doctrine/KeyValueStore/issues/91 Things done: 1. drop PHP 5.5 support: riak package, Travis, PHP itself, etc... nobody is supporting it anymore 2. replaced [riak/riak-client:dev-master](https://packagist.org/packages/riak/riak-client#dev-master) with [php-riak/riak-client:v1.0.0-alpha7](https://packagist.org/packages/php-riak/riak-client#v1.0.0-alpha7): the first doesn't support PHP 7.0+ and isn't maintained 3. changed unit test for Riak with an integration test --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

admin added the pull-request label 2026-01-23 11:35:15 +01:00

admin closed this issue

2026-01-23 11:35:15 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/KeyValueStore#114