Insert "sort" rules

[allangrds]

I noticed that there is no standard declaration of order. And that is bad because each person of the time can take a different approach or none at all.

I'd rather use the alphabetical order in var, keys, and imports because it makes it easier to find an element and understand what I'm reading.

Sort keys Sort vars Sort imports

[MarcoWorms]

Acho imports super válidos mas var e key não, pq tem casos que faz mais sentido organizar de outro jeito (quando tem property shorthand por exemplo)

Não sei se a abordagem de "quem não gostou justifique" é a melhor. Dado que vc abriu o PR seria massa vc explicar o pq isso agregaria pro styleguide.

[vcapretz]

Concordo com o Worms, ao declarar models no Sequelize por exemplo, pode fazer sentido usar uma ordem em que coisas relacionadas estejam perto.

Mesma coisa com vars, é mais viável que sejam declaradas perto de onde são utilizadas, ao invés de forçar ordem alfabética.

[grvcoelho]

Eu sou contra ordernar alfabeticamente vars e keys porque em >90% dos casos, semânticamente pode fazer sentido organizar de outra forma.

Sobre import, para mim é apenas uma decisão estilística e eu usaria somente com os imports que fazem destructuring:

import {
  add,
  filter,
  map
} from 'ramda'

[thalesmello]

Sort vars é irrelevante, já que temos apenas uma declaração por linha.

Sort imports sou a favor, mas talvez seja preciosismo.

Sort keys sou veemente contra.

Ordem pode ter significado. Exemplo:

{
  name: "company de test",
  description: "test",
  fee: 0.099,
  anticipation_model: "spot"
}

Seria muito esquisito ler um JSON desses em qualquer outra ordem, uma vez que faz sentido o nome vir primeiro (ou qualquer outra key que a pessoa programando ache importante na hora).

[allangrds]

Faz muito sentido, e concordo @thalesmello. Essa do key realmente é matadora.