Test Setup Failed
Pull Request — master (#40)
by Simon
02:15
created
src/Adaptor/SentryAdaptor.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,6 @@
 block discarded – undo
14 14
 use Sentry\State\Scope;
15 15
 use Sentry\ClientInterface;
16 16
 use Sentry\Severity;
17
-use SilverStripe\Control\Director;
18 17
 use SilverStripe\Core\Config\Configurable;
19 18
 use SilverStripe\Core\Injector\Injector;
20 19
 use SilverStripe\Core\Environment as Env;
Please login to merge, or discard this patch.
src/Log/SentryLogger.php 2 patches
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
     {
184 184
         $peak = memory_get_peak_usage(true) / 1024 / 1024;
185 185
 
186
-        return (string)round($peak, 2) . 'Mb';
186
+        return (string) round($peak, 2) . 'Mb';
187 187
     }
188 188
 
189 189
     /**
@@ -238,8 +238,7 @@  discard block
 block discarded – undo
238 238
 
239 239
         if (defined('TRUSTED_PROXY')) {
240 240
             $headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
241
-                [SS_TRUSTED_PROXY_IP_HEADER] :
242
-                null;
241
+                [SS_TRUSTED_PROXY_IP_HEADER] : null;
243 242
 
244 243
             if (!$headers) {
245 244
                 // Backwards compatible defaults
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -370,8 +370,8 @@
 block discarded – undo
370 370
 
371 371
     /**
372 372
      * Check if we are on a tagged release and if so, use that to set the client release
373
-     * @param $ref
374
-     * @return mixed
373
+     * @param string $ref
374
+     * @return string
375 375
      */
376 376
     protected static function getGitTag($ref)
377 377
     {
Please login to merge, or discard this patch.