Merge pull request #14 from andrew-demb/housekeeping

Housekeeping - cleanup readme, consistent composer.json, typo in DI service id

Author: allflame

README.md

@@ -1,23 +1,14 @@
 # Symfony Bridge for Jaeger library
 
 ## Getting started
-Register JaegerBundle like any other bundle for Symfony 4
-```
-bundles.php
-[
-...
-\Jaeger\Symfony\JaegerBundle::class => ['all' => true],
-...
-]
-```
-OR for Symfony >=2
-```
-public function registerBundles()
-    {
-        $bundles = [
-            ...
-            new \Jaeger\Symfony\JaegerBundle(),
-            ...
-        ]
-    }
+
+Register JaegerBundle like any other bundle for Symfony 4+
+
+```php
+// bundles.php
+return [
+    // ...
+    \Jaeger\Symfony\JaegerBundle::class => ['all' => true],
+    // ...
+];
 ```

composer.json

@@ -8,15 +8,13 @@
   },
   "require": {
     "php": ">=7.1",
-    "symfony/event-dispatcher": "^4.3||^5.0",
-    "symfony/console": "^4.3||^5.0",
-    "symfony/config": "^4.3||^5.0",
-    "symfony/http-kernel": "^4.3||^5.0",
-    "symfony/dependency-injection": "^4.3||^5.0",
-    "code-tool/jaeger-client-php": "^3.1"
-  },
-  "require-dev": {
-    "phpunit/phpunit": "@stable"
+    "code-tool/jaeger-client-php": "^3.1",
+    "symfony/config": "^4.3|^5.0",
+    "symfony/console": "^4.3|^5.0",
+    "symfony/dependency-injection": "^4.3|^5.0",
+    "symfony/event-dispatcher": "^4.3|^5.0",
+    "symfony/http-kernel": "^4.3|^5.0",
+    "symfony/yaml": "^4.3|^5.0"
   },
   "minimum-stability": "dev",
   "prefer-stable": true

src/Resources/services.yml

@@ -140,6 +140,9 @@ services:
     class: Jaeger\Symfony\Bridge\BackgroundSpanHandler
     arguments: ['@jaeger.tracer']
   jaeger.span.handler.gloabal:
+    alias: jaeger.span.handler.global
+    deprecated: 'The "%alias_id%" service is deprecated. Use "jaeger.span.handler.global".'
+  jaeger.span.handler.global:
     class: Jaeger\Symfony\Bridge\GlobalSpanHandler
     arguments: ['@jaeger.tracer', '@jaeger.name.generator']
   jaeger.debug.listener:
@@ -166,7 +169,7 @@ services:
       - {name: 'kernel.event_subscriber' }
   jaeger.global.span.listener:
     class: Jaeger\Symfony\Bridge\GlobalSpanListener
-    arguments: ['@jaeger.span.handler.gloabal']
+    arguments: ['@jaeger.span.handler.global']
     tags:
       - {name: 'kernel.event_subscriber' }
   jaeger.app.start.listener:
@@ -186,7 +189,7 @@ services:
       - {name: 'kernel.event_subscriber' }
   jaeger.span.handler.listener.flush:
     class: Jaeger\Symfony\Bridge\HandlerFlushListener
-    arguments: ['@jaeger.span.handler.background', '@jaeger.span.handler.gloabal']
+    arguments: ['@jaeger.span.handler.background', '@jaeger.span.handler.global']
     tags:
       - {name: 'kernel.event_subscriber' }
   jaeger.exception.listener: