Comments in between multi-line variable assignments should validate
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Comece reproduzindo os erros do PHPCS relatados para as atribuições em várias linhas e o exemplo de array; em seguida, inspecione o ruleset ou a regra que exige espaços antes do sinal de igual. A issue não menciona arquivos nem testes; considera-se concluído quando comentários entre atribuições e entradas de array alinhadas forem validados, enquanto o alinhamento existente continuar sendo exigido.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Suppose you have:
// 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). 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:
$my_array = [
'id' => 0,
// some comment about this key:
'name' => 'Foo',
// some comment about this other key
'location' => 123,
'etc' => null,
];
- Linguagem predominante
- Sem dados de linguagem
- Estrelas
- 0
- Forks
- 2
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Issues semelhantes
-
Marble Madness II is missingAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
[Request]: Snow ShotAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
ScoopInstaller/Extras#18870 ·
Mantenedores costumam responder em até 1 dia
-
type:bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
BHoM/MidasCivil_Toolkit#441 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
autopkg/hjuutilainen-recipes#345 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
Mantenedores costumam responder em até 1 dia