IIC2513-2021-2 / project

Repositorio oficial para el proyecto del curso IIC2513, período 2021-2
32 stars 1 forks source link

Eslint #32

Open Huasito-Appel opened 2 years ago

Huasito-Appel commented 2 years ago

Hola, tengo un problema al usar Eslint, este me tira error en las migraciones que genera automatico Sequelize. image image Como podemos ver me alega en al linea 32 que no uso la variable Sequelize, al ser migraciones auto-generadas preferi no tocar nada, si bien puedo arreglarlo con codigo uselles, algo asi como Sequelize = Sequelize ? esperaba tener una solucion mas pulcra.
Tambien seria ideal ignorar estos errores pero son necesarios que pase el check del linter para el merge, quedo atento 🧑‍🌾

Baelfire18 commented 2 years ago

Hola Sergio, puedes dejar la línea de esta forma

down: (queryInterface) => queryInterface.dropTable('users'),

o

down: (queryInterface) => queryInterface.bulkDelete('users', null),

De esta forma soluciona el problema de no usar la variable Sequelize y dejas todo en una sola linea tambien

La última es lo que hacemos en la seed del proyecto de ejemplo soundify.

Huasito-Appel commented 2 years ago

Gracias rey

jackyYam commented 2 years ago

Hola aprovecho a comentar otro error es eslint parecido: image El arichio tiene la siguiente forma

image

Baelfire18 commented 2 years ago

@chineseJack564 hola. Le puedes sacar la extensión .js, ya que no es necesaria al momento de importarse el archivo (es redundante especificar su terminación).

Edit: Es cierto que en el template del curso viene esa terminación, pero no hay problema en que la saquen ustedes, no afecta su rendimiento o funcionamiento