Add PHP Coding Standards Fixer in CI (#1196)

* Update composer.json

* Update .gitignore

* Update php.yml

* Apply PHPCSFixer fixes

* Switch to php-cs-fixer/shim

* Create .php-cs-fixer.dist.php

Author: jbelien

Provider/AbstractHttpProvider.php

@@ -16,8 +16,8 @@
 use Geocoder\Exception\InvalidServerResponse;
 use Geocoder\Exception\QuotaExceeded;
 use Geocoder\Provider\AbstractProvider;
-use Http\Message\MessageFactory;
 use Http\Discovery\MessageFactoryDiscovery;
+use Http\Message\MessageFactory;
 use Psr\Http\Client\ClientInterface;
 use Psr\Http\Message\RequestInterface;
 
@@ -37,10 +37,6 @@ abstract class AbstractHttpProvider extends AbstractProvider
      */
     private $messageFactory;
 
-    /**
-     * @param ClientInterface     $client
-     * @param MessageFactory|null $factory
-     */
     public function __construct(ClientInterface $client, MessageFactory $factory = null)
     {
         $this->client = $client;
@@ -50,10 +46,6 @@ public function __construct(ClientInterface $client, MessageFactory $factory = n
     /**
      * Get URL and return contents. If content is empty, an exception will be thrown.
      *
-     * @param string $url
-     *
-     * @return string
-     *
      * @throws InvalidServerResponse
      */
     protected function getUrlContents(string $url): string
@@ -63,11 +55,6 @@ protected function getUrlContents(string $url): string
         return $this->getParsedResponse($request);
     }
 
-    /**
-     * @param string $url
-     *
-     * @return RequestInterface
-     */
     protected function getRequest(string $url): RequestInterface
     {
         return $this->getMessageFactory()->createRequest('GET', $url);
@@ -76,10 +63,6 @@ protected function getRequest(string $url): RequestInterface
     /**
      * Send request and return contents. If content is empty, an exception will be thrown.
      *
-     * @param RequestInterface $request
-     *
-     * @return string
-     *
      * @throws InvalidServerResponse
      */
     protected function getParsedResponse(RequestInterface $request): string
@@ -105,17 +88,12 @@ protected function getParsedResponse(RequestInterface $request): string
 
     /**
      * Returns the HTTP adapter.
-     *
-     * @return ClientInterface
      */
     protected function getHttpClient(): ClientInterface
     {
         return $this->client;
     }
 
-    /**
-     * @return MessageFactory
-     */
     protected function getMessageFactory(): MessageFactory
     {
         return $this->messageFactory;

Tests/Provider/AbstractHttpProviderTest.php

@@ -17,9 +17,9 @@
 use Geocoder\Model\AddressCollection;
 use Geocoder\Query\GeocodeQuery;
 use Geocoder\Query\ReverseQuery;
-use Psr\Http\Client\ClientInterface;
 use Http\Mock\Client;
 use PHPUnit\Framework\TestCase;
+use Psr\Http\Client\ClientInterface;
 
 class AbstractHttpProviderTest extends TestCase
 {