Merge pull request #60 from FriendsOfPHP/update

Prepare v5

Author: lyrixx

.gitattributes

@@ -0,0 +1,69 @@
+name: CI
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+  schedule:
+    - cron: "0 0 * * MON"
+
+jobs:
+  php-cs-fixer:
+    name: Check PHP Coding Standards
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: PHP-CS-Fixer
+        uses: docker://oskarstark/php-cs-fixer-ga
+        with:
+          args: --config=.php-cs-fixer.php --diff --dry-run
+
+  phpstan:
+    name: PHPStan
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: PHPStan
+        uses: docker://oskarstark/phpstan-ga
+        with:
+          args: analyse
+
+  ci:
+    name: Test PHP ${{ matrix.php-version }} ${{ matrix.name }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        php-version: ["7.4", "8.0", "8.1"]
+        composer-flags: [""]
+        name: [""]
+        include:
+          - php-version: 7.4
+            composer-flags: "--prefer-lowest"
+            name: "(prefer lowest dependencies)"
+
+    services:
+      service-name-1:
+        image: consul
+        ports:
+          - 8500:8500
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Setup PHP, with composer and extensions
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+          extensions: xml
+
+      - name: Install Composer dependencies
+        run: composer update --prefer-dist --no-interaction ${{ matrix.composer-flags }}
+
+      - name: Run Tests
+        run: make test-phpunit

.github/workflows/ci.yml

@@ -1,4 +1,3 @@
+/.phpunit.cache/
 /composer.lock
-/build/
-/bin/consul/config/data-dir/
 /vendor/

.gitignore

@@ -0,0 +1,24 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->in(__DIR__)
+    ->append([
+        __FILE__,
+    ])
+;
+
+return (new PhpCsFixer\Config())
+    ->setRiskyAllowed(true)
+    ->setRules([
+        '@PHP74Migration' => true,
+        '@PhpCsFixer' => true,
+        '@Symfony' => true,
+        '@Symfony:risky' => true,
+        'php_unit_internal_class' => false, // From @PhpCsFixer but we don't want it
+        'php_unit_test_class_requires_covers' => false, // From @PhpCsFixer but we don't want it
+        'phpdoc_add_missing_param_annotation' => false, // From @PhpCsFixer but we don't want it
+        'ordered_class_elements' => true, // Symfony(PSR12) override the default value, but we don't want
+        'blank_line_before_statement' => true, // Symfony(PSR12) override the default value, but we don't want
+    ])
+    ->setFinder($finder)
+;

.php-cs-fixer.php

@@ -0,0 +1,23 @@
+# CHANGELOG
+
+## 5.0.0 (not released yet)
+
+Release notes:
+
+This is the first big release under friendsofphp umbrella. There are lot of BC
+Breaks, but they should be easy to fix. From now on, a particular attention will
+be given to not break the BC and to provide a nice upgrade path.
+
+* Rename package from `sensiolabs/consul-php-sdk` to `friendsofphp/consul-php-sdk`
+* Get ride of SensioLabs namespace (from `SensioLabs\Consul` to `Consul`)
+* Add typehint where possible
+* Force JSON body request where possible (now you must pass an array as body)
+* Remove the factory and almost all interfaces
+* Bump to PHP 7.4
+* Add support for missing scheme in DSN
+* Switch from Travis to GitHub Action
+* Add some internal tooling (php-cs-fixer, phpstan, phpunit, Makefile)
+
+---
+
+Previous CHANGELOGs are missing