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

Setting TypeScript y Eslint general para la solución Front, Back y Librerías. #8

Open
JoseJPR opened this issue Jul 8, 2020 · 2 comments
Labels
need decision Necesita decidirse question Further information is requested

Comments

@JoseJPR
Copy link
Member

JoseJPR commented Jul 8, 2020

@Coffee-WIP/web-site-contributors No tengo claro si hay ya un setting definido para la configuración de TypeScript y ESLint para alguno de los proyectos (front, back o tools)

Si no la hubiera me gustaría proponer:

1.- No realizar modificaciones en el setting base de TypeScript.

Para el backend y las librerías que se desarrollarán es posible tener este setting (archivo: tsconfig.json):
{ "compilerOptions": { "module": "commonjs", "declaration": true, "noImplicitAny": true, "removeComments": true, "noLib": false, "allowSyntheticDefaultImports": true, "emitDecoratorMetadata": true, "experimentalDecorators": true, "target": "es6", "sourceMap": true, "outDir": "./dist", "baseUrl": "./", "resolveJsonModule": true, "esModuleInterop": true, "lib": [ "dom", "es2017" ] }, "include": [ "src/**/*" ], "exclude": [ "node_modules", "**/__test__/*.*", "**/*.spec.ts" ] }

2.- Utilizar ESLint con la variante Airbnb (https://github.com/airbnb/javascript).

Para el backend y las librerías que se desarrollarán es posible tener este setting (archivo: .eslintrc.json):

{ "env": { "es2020": true, "node": true }, "extends": [ "airbnb-base" ], "parser": "@typescript-eslint/parser", "parserOptions": { "ecmaVersion": 11, "sourceType": "module" }, "plugins": [ "@typescript-eslint" ], "rules": { } }

@JoseJPR JoseJPR added need decision Necesita decidirse question Further information is requested labels Jul 8, 2020
@chempogonzalez
Copy link
Contributor

chempogonzalez commented Jul 8, 2020

  1. Por qué poner "noImplicitAny": false ? Ya que estamos usando Typescript, podemos aprovecharnos del tipado, no?
  2. Me parece perfecto usar la variante de Airbnb

@JoseJPR
Copy link
Member Author

JoseJPR commented Jul 9, 2020

@chempogonzalez Done! cambiado a true, fue una errata.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need decision Necesita decidirse question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants