.eslintrc rules

[alesanchezr]

@alacret @nachovz Tenemos que proponer un eslintrc que sea mas estricto con los alumnos al momento de compilar. Yo estoy proponiendo uno, revisenlo en la raiz del repositorio y avisenme que tal, me gustaria que estoy fuera un debate abierto entre los profesores.

• Debemos exigir sangrias? De 4 espacios o de dos?
• Las variables declaradas son un error o un warning?
• etc!

Aqui esta la lista de reglas: https://eslint.org/docs/rules/

Aqui esta el estandard propuesto por airbnb: https://github.com/airbnb/javascript/tree/master/react#basic-rules

[alacret]

Usemos el de airbnb o el de google y listo, no cero que haga falta discutirlo, ambos son muy adoptados y los IDE formaban el código de acuerdo al que le pongas.

[alesanchezr]

No entiendo bien, cual es la diferencia entre el de airbnb y el de google?

@nachovz tu q opinas?

[nachovz]

No sé que tan estrictos son los de Google y airbnb. En cualquier caso me parece que debemos ser más estrictos (me gusta lo que propone Alejandro). @alacret se puede editar esos lints?

[alacret]

Las diferencias no las se exactamente, pero son cosas de estilo, que si 4 espacios o 2 para indentar, son cosas triviales. Y si, tu puedes extender los settings, y sobreescribirlos:

module.exports = { "extends": "google", "rules": { "object-curly-spacing": "always" }, "parserOptions": { "ecmaVersion": 7, "sourceType": "module", "ecmaFeatures": { "jsx": true, } }, };

[nachovz]

Ok, entonces si deberíamos arrancar desde google y apretar cosas como camelCase e indentation.

[alacret]

Y TODas esas reglas vienen. Creo que sola la de los import en es7 es la que no viene por defecto

On Sat, Mar 31, 2018 at 2:02 PM Ignacio Cordoba notifications@github.com wrote:

Ok, entonces si deberíamos arrancar desde google y apretar cosas como camelCase e indentation.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/4GeeksAcademy/react-hello-flux/issues/1#issuecomment-377711699, or mute the thread https://github.com/notifications/unsubscribe-auth/AA4FbyWT2ttUPPTx-5jvnOPWuZofmoB5ks5tj8SYgaJpZM4TAq0j .

--

Ing. Angel Lacret

[alesanchezr]

Pero vieron las que yo propuse? revisen el eslintrc que esta en este repo yo me puse una a una a revisar.

[alacret]

Si ya lo revisaste yo no tengo problema en usar ese.

On Sat, Mar 31, 2018 at 11:51 PM, Alejandro Sanchez < notifications@github.com> wrote:

Pero vieron las que yo propuse? revisen el eslintrc que esta en este repo yo me puse una a una a revisar.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/4GeeksAcademy/react-hello-flux/issues/1#issuecomment-377741749, or mute the thread https://github.com/notifications/unsubscribe-auth/AA4Fb1UHsGqtnGou-SM2G-3C7tyOCUk8ks5tkE7EgaJpZM4TAq0j .

--

Ing. Angel Lacret