Question: do you enforce multiline new lines for destructuring?

Aberta
  • Algo errado?
#2,435 4 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
5/5
Tempo estimado
Mais de uma semana
Facilidade para iniciantes
25/100
Tipo de issue
Documentação
Clareza
Precisa de esclarecimento
Status de atividade
Estagnada
Stack de tecnologia
eslint, javascript
Domínio
documentation

Direção de pesquisa

Comece revisando as orientações do style-guide do AirBnB referenciadas como as regras 7.15 e 10.8, junto com os exemplos da issue sobre object destructuring e props. Compare o comportamento declarado com ESLint, Prettier e eslint-plugin-putout; o trabalho estará concluído quando houver um padrão decidido e documentado para esses casos.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

We are having a debate around whether the AirBnB guidelines for object-curly-newline (10.8) should affect destructuring or not, particularly for props.

Example:

// Good
export default function myComponent({ prop1, prop2, prop3 }) { ... }

export default function myComponent({ 
 prop1, prop2, prop3,
}) { ... }

export default function myComponent({ 
 prop1,
 prop2,
 prop3,
}) { ... }

// Is this bad?
export default function myComponent({ 
 prop1, prop2, 
 prop3, 
}) { ... }

// Good
const { env1, env2, env3 } = process.env;

// Is this bad?
const { 
  env1, env2,
  env3,
} = process.env;

We know eslint does not support this at all (and have chosen not to), so if we want to do it we have to use prettier or eslint-plugin-putout. We are curious if AirBnB standards have been defined for this circumstance? There has been many discussions around whether rules 7.15 and 10.8 were intended to affect this.

Many thanks for your time. I apologize if this was answered elsewhere, couldn't seem to find it.

Linguagem predominante
JavaScript
Estrelas
148k
Forks
26.5k
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

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de airbnb/javascript

Todas as issues de airbnb/javascript

Issues semelhantes

Mais issues de JavaScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.