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

[FEATURE] Como un desarrollador, quiero que el linter se encargue de eliminar los mensajes que mando a consola para no tener que preocuparme por subir código innecesario #9

Open
pabo99 opened this issue May 21, 2020 · 5 comments

Comments

@pabo99
Copy link
Contributor

pabo99 commented May 21, 2020

Me gustaría que si mando ciertos logs a la consola el linter los eliminara. Cosas que se pudieran eliminar son:

en PHP:

  • error_log
  • print_r
  • print

en JavaScript/TypeScript:

  • console.log
  • console.error

Suponiendo que alguno de estos si fuera necesario, que se pueda agregar una bandera tal como se hizo con los IDs en los archivos de Vue

@carlosabcs
Copy link
Member

Y si pudiera haber una banderita tipo -- preserve-logging que me permita no borrar esos logs. Creo que también sería bueno.

@lhchavez
Copy link
Member

esa banderita sería no-factible :( la idea es que si se envía algo a GitHub debería no tener logging statements de debugging.

@carlosabcs
Copy link
Member

Oh claro, el logging solo sería para poder ejecutar stuff/lint.sh -- preserve-logging, en local. Los PRs irían sin la bandera.

@lhchavez
Copy link
Member

me gustaría continuar con tener cero banderas de personalización para el linter si es posible.

@lhchavez
Copy link
Member

habiendo dicho eso, algo que sí podríamos hacer es que ahorita es posible elegir un subconjunto de linters (ejemplo: lint.sh --linters=api fix). podríamos agregar una manera de elegir todos excepto uno (se me ocurre lint.sh --exclude-linters=api fix).

para el linter de eliminar logs de php posiblemente se implementaría como uno separado (así que sí sería posible excluirlo), pero para el de javascript sería más complicado porque se puede incluir en el linter que ya tenemos.

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

No branches or pull requests

3 participants