Update nikolaposa/version to version 4

.gitignore

@@ -4,3 +4,4 @@
 /bin/
 /composer.lock
 /vendor/
+/docker-compose.yml

.travis.yml

@@ -5,9 +5,9 @@ cache:
         - $HOME/.composer/cache/files
 
 addons:
-  apt:
-    packages:
-    - jq
+    apt:
+        packages:
+            - jq
 
 env:
     global:
@@ -18,13 +18,14 @@ matrix:
     fast_finish: true
     include:
           # Minimum supported dependencies with the latest and oldest PHP version
-        - php: 7.4
+        - php: 7.2
           env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest"
-        - php: 7.1
+        - php: 7.3
+          env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest"
+        - php: 7.4
           env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest"
 
           # Test the latest stable release
-        - php: 7.1
         - php: 7.2
         - php: 7.3
         - php: 7.4
@@ -69,8 +70,8 @@ script:
     - ./vendor/bin/phpunit ${PHPUNIT_FLAGS}
 
 after_script:
-  - |
-    if [ "$COVERAGE" = true ] && [ -e coverage.clover ]; then
-      wget https://scrutinizer-ci.com/ocular.phar
-      php ocular.phar code-coverage:upload --format=php-clover coverage.clover
-    fi
+    - |
+      if [ "$COVERAGE" = true ] && [ -e coverage.clover ]; then
+        wget https://scrutinizer-ci.com/ocular.phar
+        php ocular.phar code-coverage:upload --format=php-clover coverage.clover
+      fi

Formatter/GitDescribeFormatter.php

@@ -2,6 +2,7 @@
 
 namespace Shivas\VersioningBundle\Formatter;
 
+use Version\Extension\PreRelease;
 use Version\Version;
 
 /**
@@ -15,44 +16,29 @@ class GitDescribeFormatter implements FormatterInterface
      * @param  Version $version
      * @return Version
      */
-    public function format(Version $version)
+    public function format(Version $version): Version
     {
-        if (preg_match('/^(\d+)-g([a-fA-F0-9]{7,40})(-dirty)?$/', $version->getPreRelease(), $matches)) {
-            if ((int) $matches[1] != 0) {
-                // we are not on TAG commit, add "dev" and git commit hash as pre release part
-                $version = $version->withPreRelease('dev.' . $matches[2]);
-            } else {
-                $version = $this->clearPreRelease($version);
-            }
+        $preRelease = $version->getPreRelease();
+        if (!$preRelease instanceof PreRelease) {
+            return $version;
+        }
+
+        if (preg_match('/^(\d+)-g([a-fA-F0-9]{7,40})(-dirty)?$/', $preRelease->toString(), $matches)) {
+            $withPreRelease = (int) $matches[1] != 0 ? sprintf('dev.%s',  $matches[2]) : null;
+            $version = $version->withPreRelease($withPreRelease);
         }
 
-        if (preg_match('/^(.*)-(\d+)-g([a-fA-F0-9]{7,40})(-dirty)?$/', $version->getPreRelease(), $matches)) {
+        if (preg_match('/^(.*)-(\d+)-g([a-fA-F0-9]{7,40})(-dirty)?$/', $preRelease->toString(), $matches)) {
             if ((int) $matches[2] != 0) {
-                // we are not on TAG commit, add "dev" and git commit hash as pre release part
-                if (empty($matches[1])) {
-                    $version = $version->withPreRelease('dev.' . $matches[3]);
-                } else {
-                    $version = $version->withPreRelease(trim($matches[1], '-') . '.dev.' . $matches[3]);
-                }
+                // if we are not on TAG commit, add "dev" and git commit hash as pre release part
+                $withPreRelease = empty($matches[1]) ? sprintf('dev.%s',  $matches[3]) : sprintf('%s.dev.%s', trim($matches[1], '-'),  $matches[3]);
+                $version = $version->withPreRelease($withPreRelease);
             } else {
-                if (empty($matches[1])) {
-                    $version = $this->clearPreRelease($version);
-                } else {
-                    $version = $version->withPreRelease(trim($matches[1], '-'));
-                }
+                $withPreRelease = empty($matches[1]) ? null : trim($matches[1], '-');
+                $version = $version->withPreRelease($withPreRelease);
             }
         }
 
         return $version;
     }
-
-    private function clearPreRelease(Version $version): Version
-    {
-        if (class_exists(\Version\Metadata\PreRelease::class)) {
-            // we cannot use null with nikolaposa/version 2.2
-            return $version->withPreRelease(\Version\Metadata\PreRelease::createEmpty());
-        } else {
-            return $version->withPreRelease(null);
-        }
-    }
 }

Service/VersionManager.php

@@ -8,7 +8,7 @@
 use Shivas\VersioningBundle\Provider\ProviderInterface;
 use Shivas\VersioningBundle\Writer\WriterInterface;
 use Symfony\Component\Cache\Adapter\AdapterInterface;
-use Version\Exception\InvalidVersionStringException;
+use Version\Exception\InvalidVersionString;
 use Version\Version;
 
 /**
@@ -178,7 +178,7 @@ public function getVersionFromProvider()
             }
 
             return $version;
-        } catch (InvalidVersionStringException $e) {
+        } catch (InvalidVersionString $e) {
             throw new RuntimeException(get_class($provider) . ' returned an invalid version');
         }
     }

composer.json

@@ -17,8 +17,8 @@
         "wiki": "https://github.com/shivas/versioning-bundle/wiki"
     },
     "require": {
-        "php": "^7.1",
-        "nikolaposa/version": "^2.2 || ^3",
+        "php": "^7.2",
+        "nikolaposa/version": "^4",
         "symfony/console": "^3.4 || ^4 || ^5",
         "symfony/framework-bundle": "^3.4 || ^4 || ^5",
         "symfony/process": "^3.4 || ^4 || ^5"