Resolver lint warnings about sme-filadacreche HOT 3 CLOSED

De acordo @victorsenam! O problema é que alguns desses warnings são falsos, especialmente os 'Router' is defined but never used, relativos ao router.

Da última vez que tentei limpar todas essas variáveis não usadas eu acabei quebrando a aplicação, então precisamos entender pq ela depende de variáveis que o linter entende como não usadas.

from sme-filadacreche.

@victorsenam fiz merge no #46 (review)

Resolveu vários erros. Você se dá melhor com o linter que eu 😄.

Agora eu vejo só:

Line 13:  Missing radix parameter  radix
Line 14:  Missing radix parameter  radix

./src/containers/Results/Results.js
Line 15:  Expected '===' and instead saw '=='  eqeqeq

./src/containers/DateOfBirthForm/SelectDate.js
Line 48:  Missing radix parameter  radix
Line 49:  Missing radix parameter  radix

Esse Line 15: Expected '===' and instead saw '==' eqeqeq eu acho que é melhor deixar assim pelo menos por enquanto, por não termos garantia se o dado virá da API como string ou integer.

from sme-filadacreche.

Beleza, se a gente tem medo desse === ele é um bom caso da dar disable. Essa coisa de falhar ao apagar o Router, eu tinha mudado, por exemplo, a linha
import { BrowserRouter as Router, Route, Link, HashRouter } from 'react-router-dom';
para
import { BrowserRouter as Route, Link, HashRouter } from 'react-router-dom';
em vez de
import { Route, Link, HashRouter } from 'react-router-dom';
e isso tinha me confundido bastante, achei que não estava funcionando, mas é que na real eu tinha feito errado. Talvez você tenha cometido o mesmo erro quando pensou que não fosse funcionar arrumar os warnings.

from sme-filadacreche.