Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vscode config #422

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Vscode config #422

wants to merge 3 commits into from

Conversation

gmq
Copy link
Member

@gmq gmq commented Nov 18, 2022

Agrega carpeta .vscode con extensiones recomendadas al abrir un proyecto. Además agrega rulers para indicar el largo de linea configurado en eslint y rubocop.

Se podrían agregar más configuraciones pero como las de workspace sobreescriben las personales, hay mucha posibilidad de cambiar algo que le incomode a alguien.

@ldlsegovia
Copy link
Member

Agrega carpeta .vscode con extensiones recomendadas al abrir un proyecto. Además agrega rulers para indicar el largo de linea configurado en eslint y rubocop.

Se podrían agregar más configuraciones pero como las de workspace sobreescriben las personales, hay mucha posibilidad de cambiar algo que le incomode a alguien.

Igual son recomendaciones, no? podríamos meterle más y que la gente decida.

@gmq
Copy link
Member Author

gmq commented Nov 18, 2022

Igual son recomendaciones, no? podríamos meterle más y que la gente decida.

Las extensiones si, pero lo que está en settings.json se considera como settings del workspace y se aplican por encima de las que tengas configuradas en general. El mayor problema es que si cambias en tu editor las del workspace para cambiar algo que no te gustó, va a quedar molestando por siempre el cambio en git

@difernandez
Copy link
Contributor

Igual son recomendaciones, no? podríamos meterle más y que la gente decida.

Las extensiones si, pero lo que está en settings.json se considera como settings del workspace y se aplican por encima de las que tengas configuradas en general. El mayor problema es que si cambias en tu editor las del workspace para cambiar algo que no te gustó, va a quedar molestando por siempre el cambio en git

Igual revisé, y al parecer solo un proyecto tiene en github la carpeta .vscode, y en un repo de ejemplo:
image

Creo que hay varios proyectos que lo tienen en el gitignore, y/o varias personas (como yo) que lo tienen en su gitignore global. Con eso, me entró la duda de si sería tan útil o habría que hacer algo más 🤔

@gmq
Copy link
Member Author

gmq commented Nov 18, 2022

O sea, si el archivo viene en el repo, aunque esté en el gitignore global igual se va a usar 🤔

@difernandez
Copy link
Contributor

pero si está en el gitignore global o del proyecto, no se va a subir a github, entonces esto le serviría solo a la persona que inicialmente generó el proyecto con potassium, o na que ver?

@gmq
Copy link
Member Author

gmq commented Nov 18, 2022

Cierto 🤔
Se podría agregar !.vscode al .gitignore del proyecto para que sobreescriba el .gitignore global

@gmq
Copy link
Member Author

gmq commented Nov 18, 2022

O sacar el settings.json y dejar explicitamente solo !.vscode/extensions.json en el .gitignore, por si hay gente que usa los workspace settings para ordenarse

@difernandez
Copy link
Contributor

Cacha, no sabía que se podía hacer eso con !. Me gusta lo último que pusiste!

@gmq
Copy link
Member Author

gmq commented Nov 18, 2022

Estaba viendo y en Angular incluyen archivos tipo ejemplo https://github.com/angular/angular/tree/main/.vscode#usage, para no sobreescribir lo real

Comment on lines 2 to 5
"editor.rulers": [
100,
120
],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quizás en vez de poner dos, mejor hacerlo por lenguaje?

  "[ruby]": {
    "editor.rulers": [
      100
    ],
  },
  "[vue]": {
    "editor.rulers": [
      120
    ],
  },
  "[javascript]": {
    "editor.rulers": [
      120
    ]
  "[typescript]": {
    "editor.rulers": [
      120
    ]
  },

@difernandez
Copy link
Contributor

Estaba viendo y en Angular incluyen archivos tipo ejemplo https://github.com/angular/angular/tree/main/.vscode#usage, para no sobreescribir lo real

Está bueno eso pa los settings, like

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants