Skip to content

Commit

Permalink
fixed psalm static errors
Browse files Browse the repository at this point in the history
  • Loading branch information
glo71317 committed Oct 13, 2023
1 parent 19241c7 commit 83f39f2
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 2 deletions.
4 changes: 4 additions & 0 deletions src/Figlet/Figlet.php
Original file line number Diff line number Diff line change
Expand Up @@ -832,6 +832,10 @@ protected function _smushAmount()
if (empty($leftChar) || $leftChar === ' ') {
$amount++;
} elseif (! empty($rightChar)) {
/**

Check failure on line 835 in src/Figlet/Figlet.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Whitespace found at end of line
* @psalm-suppress RedundantConditionGivenDocblockType Avoid behavior BC break

Check failure on line 836 in src/Figlet/Figlet.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Whitespace found at end of line
* @psalm-suppress MixedArgument
*/
if ($this->_smushem($leftChar, $rightChar) !== null) {
$amount++;
}
Expand Down
5 changes: 4 additions & 1 deletion src/Table/DecoratorManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,10 @@ class DecoratorManager extends AbstractPluginManager
'zendtexttabledecoratorunicode' => Decorator\Unicode::class,
];

/** @var FactoriesConfigurationType */
/**

Check failure on line 49 in src/Table/DecoratorManager.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Whitespace found at end of line
* @var FactoriesConfigurationType

Check failure on line 50 in src/Table/DecoratorManager.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Whitespace found at end of line
* @psalm-suppress NonInvariantDocblockPropertyType Avoid behavior BC break
*/
protected $factories = [
Decorator\Ascii::class => InvokableFactory::class,
Decorator\Unicode::class => InvokableFactory::class,
Expand Down
2 changes: 1 addition & 1 deletion src/Table/Table.php
Original file line number Diff line number Diff line change
Expand Up @@ -498,7 +498,7 @@ public function render()

/**
* Magic method which returns the rendered table
*
* @psalm-suppress MethodSignatureMustProvideReturnType Avoid behavior BC break

Check failure on line 501 in src/Table/Table.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Expected 1 line between description and annotations, found 0.

Check failure on line 501 in src/Table/Table.php

View workflow job for this annotation

GitHub Actions / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

Whitespace found at end of line
* @return string
*/
public function __toString()
Expand Down
1 change: 1 addition & 0 deletions test/MultiByteTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

/**
* @group Laminas_Text
* @psalm-suppress DeprecatedMethod
*/
class MultiByteTest extends TestCase
{
Expand Down

0 comments on commit 83f39f2

Please sign in to comment.