codenautas / qa-control

QA control of things that nobody controls
MIT License
1 stars 0 forks source link

Crear la versión (package-version) 0.2.0 #52

Closed emilioplatzer closed 8 years ago

emilioplatzer commented 8 years ago

En la versión 0.2.0:

Para aceptar ecmaScript 6 hay que poner en el package.json:

  "jshintConfig": {
    "asi": false,
    "forin": true,
    "curly": true,
    "esversion": 6
  },
  "eslintConfig": {
    "env": {
      "node": false
    },
    "rules": {
      "strict": 0,
      "no-console": 1,
      "no-unused-vars": 1
    },
    "parserOptions": {
      "ecmaVersion": 6
    }
  },

para probar usar el proyecto pizarra

diegoefe commented 8 years ago

La opción "esversion" de jshint existe a partir de la versión 2.9.0. Al actualizar qa-control, tengo warnings para los fixtures que corren versión 0.1.3 (al menos): 'Incompatible values for the \'strict\' and \'globalstrict\' linting options...' Alguna sugerencia?

emilioplatzer commented 8 years ago

A) Tener en cada versión una White List de warnings que no nos joden

B) pasarle más opciones a eslint para apagar ese control El mar. 22, 2016 15:57, "Diego Florio" notifications@github.com escribió:

La opción "esversion" de jshint existe a partir de la versión 2.9.0. Al actualizar qa-control, tengo warnings para los fixtures que corren versión 0.1.3 (al menos): 'Incompatible values for the \'strict\' and \'globalstrict\' linting options...' Alguna sugerencia?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/52#issuecomment-199962716

diegoefe commented 8 years ago

Es jshint, no eslint. Si uso la versión vieja, no tengo esversion y falla cuando encuentra la opción Si uso la nueva, no acepta globalstrict=true y no encontré como apagar un error de opción (no de código)

emilioplatzer commented 8 years ago

Usa la nueva y A) filtrá la salida o B) agregá como configuración inicial o como opción (de alguna de las múltiples maneras posibles) que no controle eso El mar. 22, 2016 17:09, "Diego Florio" notifications@github.com escribió:

Es jshint, no eslint. Si uso la versión vieja, no tengo esversion y falla cuando encuentra la opción Si uso la nueva, no acepta globalstrict=true y no encontré como apagar un error de opción (no de código)

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/52#issuecomment-199998527

diegoefe commented 8 years ago

A) No se puede porque fallan todos los first_lines (excepto los de 0.2.0) B) No se pueden deshabilitar los code: 'E...', que son errores, no warnings, en especial E059 (conflicto entre los valores de strict y gobalstrict), o aún no encontré la manera Todos los tests de stable-project (0.1.3), que son la mayoría, tienen este problema

emilioplatzer commented 8 years ago

A se puede siempre. La lista de warnings que recibís la pasás por un filtro que que ese warning y ya no hay warnings. El mar. 22, 2016 6:21 PM, "Diego Florio" notifications@github.com escribió:

A) No se puede porque fallan todos los first_lines (excepto los de 0.2.0) B) No se pueden deshabilitar los code: 'E...' errores, no warnings, en especial E059 (conflicto entre los valores de strict y gobalstrict), o aún no encontré la manera Todos los tests de stable-project (0.1.3), que son la mayoría, tienen este problema

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/52#issuecomment-200035450

emilioplatzer commented 8 years ago

Veo que la opción B se puede basar en --config, se pueden recopilar las opciones, agregar las que sea, guardar eso en un archivo temporal y pasárselo con --config

Usando el API (http://jshint.com/docs/api/) es más fácil porque el objeto con las opciones se pasa directamente

2016-03-22 18:56 GMT-03:00 Emilio Platzer emilioplatzer@gmail.com:

A se puede siempre. La lista de warnings que recibís la pasás por un filtro que que ese warning y ya no hay warnings. El mar. 22, 2016 6:21 PM, "Diego Florio" notifications@github.com escribió:

A) No se puede porque fallan todos los first_lines (excepto los de 0.2.0) B) No se pueden deshabilitar los code: 'E...' errores, no warnings, en especial E059 (conflicto entre los valores de strict y gobalstrict), o aún no encontré la manera Todos los tests de stable-project (0.1.3), que son la mayoría, tienen este problema

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/52#issuecomment-200035450

diegoefe commented 8 years ago

Ok, me fijaré, pero vos estás seguro de que los E? pueden anularse con opciones?

emilioplatzer commented 8 years ago

No, seguro no estoy. Hay que mirar. Si no lo encontrás en las opciones avisáme

On Tue, Mar 22, 2016 at 7:23 PM, Diego Florio notifications@github.com wrote:

Ok, me fijaré, pero vos estás seguro de que los E? pueden anularse con opciones?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/52#issuecomment-200059812

diegoefe commented 8 years ago

Ok, te aviso.