Skip to content

Commit

Permalink
Remove DocBlockLineLengthFixer, as not used and often breaking, bette…
Browse files Browse the repository at this point in the history
…r handle manually
  • Loading branch information
TomasVotruba committed Jun 19, 2023
1 parent 4ddb721 commit e6f9e61
Show file tree
Hide file tree
Showing 17 changed files with 4 additions and 685 deletions.
42 changes: 4 additions & 38 deletions docs/rules_overview.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# 13 Rules Overview
# 12 Rules Overview

## ArrayListItemNewlineFixer

Expand Down Expand Up @@ -43,42 +43,6 @@ Strict type declaration has to be followed by empty line

<br>

## DocBlockLineLengthFixer

Docblock lenght should fit expected width

:wrench: **configure it!**

- class: [`Symplify\CodingStandard\Fixer\LineLength\DocBlockLineLengthFixer`](../src/Fixer/LineLength/DocBlockLineLengthFixer.php)

```php
<?php

declare(strict_types=1);

use Symplify\CodingStandard\Fixer\LineLength\DocBlockLineLengthFixer;
use Symplify\EasyCodingStandard\Config\ECSConfig;

return static function (ECSConfig $ecsConfig): void {
$ecsConfig->ruleWithConfiguration(DocBlockLineLengthFixer::class, [DocBlockLineLengthFixer::LINE_LENGTH => 40]);
};
```


```diff
/**
- * Super long doc block description
+ * Super long doc
+ * block description
*/
function some()
{
}
```

<br>

## LineLengthFixer

Array items, method parameters, method call arguments, new arguments should be on same/standalone line to fit line length.
Expand All @@ -96,7 +60,9 @@ use Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer;
use Symplify\EasyCodingStandard\Config\ECSConfig;

return static function (ECSConfig $ecsConfig): void {
$ecsConfig->ruleWithConfiguration(LineLengthFixer::class, [LineLengthFixer::LINE_LENGTH => 40]);
$ecsConfig->ruleWithConfiguration(LineLengthFixer::class, [
LineLengthFixer::LINE_LENGTH => 40,
]);
};
```

Expand Down
235 changes: 0 additions & 235 deletions src/Fixer/LineLength/DocBlockLineLengthFixer.php

This file was deleted.

45 changes: 0 additions & 45 deletions src/ValueObject/DocBlockLines.php

This file was deleted.

Loading

0 comments on commit e6f9e61

Please sign in to comment.