Skip to content

Commit

Permalink
Enable more PHP-CS-Fixer rules for PHPDoc (#864)
Browse files Browse the repository at this point in the history
* Use phpdoc_order rule

* Use phpdoc_to_comment rule

* Use align_multiline_comment rule

* Use no_blank_lines_after_phpdoc rule

* Use no_empty_phpdoc rule
  • Loading branch information
ob-stripe committed Feb 10, 2020
1 parent c0ef5be commit 4ee75f6
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 12 deletions.
4 changes: 4 additions & 0 deletions .php_cs.dist
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,9 @@ return PhpCsFixer\Config::create()
'no_alternative_syntax' => true,
'no_binary_string' => true,
'no_blank_lines_after_class_opening' => true,
'no_blank_lines_after_phpdoc' => true,
'no_empty_comment' => true,
'no_empty_phpdoc' => true,
'no_empty_statement' => true,
'no_extra_blank_lines' => true,
'no_homoglyph_names' => true,
Expand Down Expand Up @@ -91,10 +93,12 @@ return PhpCsFixer\Config::create()
'phpdoc_no_alias_tag' => true,
'phpdoc_no_empty_return' => true,
'phpdoc_no_useless_inheritdoc' => true,
'phpdoc_order' => true,
'phpdoc_return_self_reference' => true,
'phpdoc_scalar' => true,
'phpdoc_separation' => true,
'phpdoc_single_line_var_spacing' => true,
'phpdoc_to_comment' => true,
'phpdoc_trim' => true,
'phpdoc_trim_consecutive_blank_line_separation' => true,
'phpdoc_types' => true,
Expand Down
16 changes: 8 additions & 8 deletions lib/ApiRequestor.php
Original file line number Diff line number Diff line change
Expand Up @@ -104,9 +104,9 @@ private static function _encodeObjects($d)
* @param array|null $params
* @param array|null $headers
*
* @return array tuple containing (ApiReponse, API key)
*
* @throws Exception\ApiErrorException
*
* @return array tuple containing (ApiReponse, API key)
*/
public function request($method, $url, $params = null, $headers = null)
{
Expand Down Expand Up @@ -295,10 +295,10 @@ private static function _defaultHeaders($apiKey, $clientInfo = null)
* @param array $params
* @param array $headers
*
* @return array
*
* @throws Exception\AuthenticationException
* @throws Exception\ApiConnectionException
*
* @return array
*/
private function _requestRaw($method, $url, $params, $headers)
{
Expand Down Expand Up @@ -384,9 +384,9 @@ private function _requestRaw($method, $url, $params, $headers)
/**
* @param resource $resource
*
* @return \CURLFile|string
*
* @throws Exception\InvalidArgumentException
*
* @return \CURLFile|string
*/
private function _processResourceParam($resource)
{
Expand All @@ -412,10 +412,10 @@ private function _processResourceParam($resource)
* @param int $rcode
* @param array $rheaders
*
* @return array
*
* @throws Exception\UnexpectedValueException
* @throws Exception\ApiErrorException
*
* @return array
*/
private function _interpretResponse($rbody, $rcode, $rheaders)
{
Expand Down
4 changes: 2 additions & 2 deletions lib/ApiResource.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,9 @@ public function __set($k, $v)
}

/**
* @return ApiResource The refreshed resource.
*
* @throws Exception\ApiErrorException
*
* @return ApiResource The refreshed resource.
*/
public function refresh()
{
Expand Down
4 changes: 2 additions & 2 deletions lib/Webhook.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ abstract class Webhook
* @param int $tolerance maximum difference allowed between the header's
* timestamp and the current time
*
* @return Event the Event instance
*
* @throws Exception\UnexpectedValueException if the payload is not valid JSON,
* @throws Exception\SignatureVerificationException if the verification fails.
*
* @return Event the Event instance
*/
public static function constructEvent($payload, $sigHeader, $secret, $tolerance = self::DEFAULT_TOLERANCE)
{
Expand Down

0 comments on commit 4ee75f6

Please sign in to comment.