search

platanus / guides

guides!
27 stars 13 forks source link

.editorconfig #66

Closed ldlsegovia closed 9 years ago

ldlsegovia commented 9 years ago

El .editorconfig, pienso es algo en lo que deberíamos ponernos de acuerdo. Se vuelve molesto por ej cuando uno tiene los .js con tabs y otro con espacios. A la hora de hacer commit, muchas veces quedan archivos mal indentados, con tabs y espacios mezclados si uno no se preocupó en dejar eso consistente.

Lo otro es el SublimeLinter (con jshint y no se qué más) y Trailing Spaces. Creo que todos deberíamos tenerlo instalado para que, quien lo tiene instalado, no tenga que ver luces de colores por los ";" faltantes o espacios de más. (Sin mencionar que el código queda más ordenado)

Como decía en el comentario sobre Coffe... A mi la verdad no me importa mucho si usamos Coffe o no Coffe, 2 tabs o 4 espacios. Si me interesa más que todos usemos las mismas herramientas.

bunzli commented 9 years ago

+1

agustinf commented 9 years ago

Yo no tengo tanta opinión en esto, pero como Leandro, solo quiero que alguien sea Pinochet y decida por los demás.

juliogarciag commented 9 years ago

Bueno, yo opino que no me gustan los tabs (se ven distintos al navegar en github, en mi computadora y no es un caracter que pueda usar así nomás en un formulario en un sitio web). Por otro lado, sería bueno tener un .editorconfig en la plantilla de rails de platanus. Lo del .editorconfig me parece suficiente porque hace lo mismo que Trailling Spaces y porque .editorconfig tiene plugins para varios editores de textos.

Sobre lo que sí tengo mucha opinión pero podría dar el brazo a torcer en pos de estandarizar es sobre el SublimeLinter sobre todo al hablar de los puntos y comas porque a mí me parecería bueno sacarle provecho a la ASI de javascript. Sobre ese asunto creo que se podía discutir por horas como estas personas, pero igual no tendría problemas en poner puntos y comas luego de cada statement.

bunzli commented 9 years ago

Mi preferencia está por lo más estándar del open-source. Si hay empate se tira una moneda (o random.org). Pero lo que más me importa es que las preferencias estén guardadas en un archivo de modo que automáticamente yo haga caso a los estándares sin siquiera saberlo.

--  Jaime Bünzli Platan.us

On December 11, 2014 at 5:14:46 PM, Julio García (notifications@github.com) wrote:

Bueno, yo opino que no me gustan los tabs (se ven distintos al navegar en github, en mi computadora y no es un caracter que pueda usar así nomás en un formulario en un sitio web). Por otro lado, sería bueno tener un .editorconfig en la plantilla de rails de platanus. Lo del .editorconfig me parece suficiente porque hace lo mismo que Trailling Spaces y porque .editorconfig tiene plugins para varios editores de textos.

Sobre lo que sí tengo mucha opinión pero podría dar el brazo a torcer en pos de estandarizar es sobre el SublimeLinter sobre todo al hablar de los puntos y comas porque a mí me parecería bueno sacarle provecho a la ASI de javascript. Sobre ese asunto creo que se podía discutir por horas como estas personas, pero igual no tendría problemas en poner puntos y comas luego de cada statement.

— Reply to this email directly or view it on GitHub.

blackjid commented 9 years ago

creo que son dos temas distintos asi que hice otro issue #70 por si alguien tiene comentarios con ese tema.

sobre .editorconfig, me parece ultra necesario, creo que ni voy a poner este issue en la columna discucion, y voy a ser mas taliban esta vez... :)

Teniendo en cuente estas caracterizticas

When opening a file, EditorConfig plugins look for a file named .editorconfig in the directory of the opened file and in every parent directory. A search for .editorconfig files will stop if the root filepath is reached or an EditorConfig file with root=true is found.

EditorConfig files are read top to bottom and the closest EditorConfig files are read last. Properties from matching EditorConfig sections are applied in the order they were read, so properties in closer files take precedence.

Mi idea seria la siguiente:

bunzli commented 9 years ago

Negro, y si queremos cambiar algo al editorconfig en el camino, se va a actualizar al correr boxen de nuevo? On Dec 12, 2014 8:35 AM, "Juan Ignacio Donoso" notifications@github.com wrote:

creo que son dos temas distintos asi que hice otro issue #70 https://github.com/platanus/guides/issues/70 por si alguien tiene comentarios con ese tema.

sobre .editorconfig, me parece ultra necesario, creo que ni voy a poner este issue en la columna discucion, y voy a ser mas taliban esta vez... :)

Teniendo en cuente estas caracterizticas

When opening a file, EditorConfig plugins look for a file named .editorconfig in the directory of the opened file and in every parent directory. A search for .editorconfig files will stop if the root filepath is reached or an EditorConfig file with root=true is found.

EditorConfig files are read top to bottom and the closest EditorConfig files are read last. Properties from matching EditorConfig sections are applied in the order they were read, so properties in closer files take precedence.

Mi idea seria la siguiente:

  • Agregar en boxen un .editorconfig con root=true y los estandares que vamos a user, que se grabe en la carpeta src (boxen::config::src) ( podemos ver una alternativas si no todos usan esa carpeta para poner sus projectos. Puede ser el $HOME tambien.
  • Boxen debiera installar automaticamente el plugin editorconfig para el editor de preferencia. O puede ser varios editores..
  • La platanilla rails debiera tener un .editorconfig con root=false
  • La futura platanilla ionic debiera tener un .editorconfig con root=false

— Reply to this email directly or view it on GitHub https://github.com/platanus/guides/issues/66#issuecomment-66762198.

blackjid commented 9 years ago

por su puesto..