godaddy-wordpress / godaddy-wordpress/php-coding-standards
Comments in between multi-line variable assignments should validate
- 主要言語
- 言語のデータがありません
- スター
- 0
- フォーク
- 2
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Suppose you have:
```php
// group 1
$foo = 1;
$barbaz = 'variable';
$boo = null;
$anothervar = new \stdClass();
// group 2
// some comment about $foo
$foo = 1;
// some comment about $barbaz
$barbaz = 'variable';
$boo = null;
// some comment about $anothervar
$anothervar = new \stdClass();
```
With the current PHPCS ruleset group 1 would pass but group 2 would not and generate errors.
In my opinion this should be legal for single line small comments to exist in a multi-line group.
I am not sure though that this would be supported by a PHPCS rule. Probably not? Maybe we could turn off spaces before equals sign or set to severity 0. The thing is this pattern often exists in our codebase (for example [here](https://github.com/skyverge/wc-plugins/pull/3359/files#r351621661)). If we have a reason why we allow and like the alignment in group 1, we should allow here as well.
By the way this would also apply to values assignment in arrays:
```php
$my_array = [
'id' => 0,
// some comment about this key:
'name' => 'Foo',
// some comment about this other key
'location' => 123,
'etc' => null,
];
```
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
評価
この issue はまだ評価されていません。