mozilla/testpilot

Refine .eslintrc rules for node vs browser JS

Ouverte

#3 228 ouverte le 10 janv. 2018

 (6 commentaires) (0 réaction) (0 personne assignée)FreeMarker (108 forks)auto 404
dev papercutgood first issueskill:js

Métriques du dépôt

Stars
 (250 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

As @pdehaan mentions:

What about just specifying the ESLint environment in the gulpfile.babel.js and webpack.config.js, a la /* eslint-env node, browser, whatever */. Or if we want to set the node env for a directory, we could just use a nested .eslintrc file (for tests or gulp tasks or whatnot).

We've got a pretty broad set of globals and envs covering all of our JS for linting, right now. But, some JS is for the build chain in node and some are for running in the browser. Might be nice to use per-directory & file glob eslint overrides to further focus what's allowed for various source files.

Guide contributeur