1
0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-11-29 23:58:06 +01:00
Commit Graph

16 Commits

Author SHA1 Message Date
Mark Stacey
04ae92a2c0
Lint .json files (#6852)
We've been using the `eslint-plugin-json` plugin for some time, but we
haven't been visiting `.json` files in the lint script. The lint script
has now been updated to incude `.json` files, which means any invalid
JSON will result in a lint error.

Unfortunately this JSON plugin doesn't seem to apply the other eslint
rules (such as `key-spacing`) to the JSON files. I wasn't able to find a
way to get that to work. Instead I manually auto-formatted each of the
locale `message.json` files, which fixed many whitespace
inconsistencies.

The `states.json` file was deleted completely, as it appears to be
unused. It wasn't a valid JSON file anyway, it was JavaScript. It looks
like a `states.js` file is automatically generated, but an old copy was
accidentally saved as `states.json` and included in the repo.

Many duplicate key errors were found and fixed in the
`development/states/` JSON files.

`package-lock.json` was added to `.eslintignore` because it was very
slow to lint, and linting it doesn't provide much value.
2019-07-15 11:03:22 -03:00
kumavis
48047f8194 mascara - remove from project (#6283) 2019-03-12 11:17:21 -02:30
Bruno Barbieri
ae8626eceb
Merge pull request #5797 from MetaMask/trezor-v6
Add Firefox and Brave support for Trezor
2018-11-21 14:45:55 -05:00
Whymarrh Whitby
e68008edf4 Remove first-time and navigate-txs integration tests 2018-11-20 14:04:50 -03:30
brunobar79
13820b6cc1 fix conflicts 2018-10-16 14:01:54 -04:00
Bruno Barbieri
ccab4ee1a4 tests - integration - Add Drizzle tests (#5467)
* added drizzle app for testing

* working

* clean up

* clean up script

* make build step required

* add drizzle-tests to .eslintignore

* clean up drizzle run script

* lint

* use truffle unbox

* undo eslintignore changes

* revert change

* dont use global

* dont need this steps

* use the new account flow

* restore package-lock.json
2018-10-10 01:12:43 -04:00
Whymarrh Whitby
bd09f01d7e eslintignore: Add test coverage dir to eslintignore 2018-09-24 17:32:00 -02:30
brunobar79
728aeb9344 add app/vendor dir to eslintignore 2018-08-09 23:38:01 -04:00
kumavis
b3b33b4697
Update .eslintignore 2018-07-03 12:43:14 -07:00
brunobar79
61fcec78f8 update eslint rules and paths 2018-07-02 18:35:30 -04:00
Jason Clark
abefcc9612 more eslint fixes 2017-11-25 15:57:54 -07:00
Jason Clark
6f5c32b7ce adding blockies lib to eslint ignore so it doesn't blow up the style checker and fail the build 2017-11-25 15:52:46 -07:00
Thomas Huang
73ee4bdec3 Ignore tests bundle, jquery, abd helpers 2017-05-04 14:33:29 -07:00
Kevin Serrano
a07e5826ad
Convert conversion util to pure json. 2016-11-18 16:49:39 -08:00
Kevin Serrano
7d1b2db87e linting and ignoring. 2016-08-16 10:48:31 -07:00
Dan Finlay
6658bea8d4 Implement some cross-browser practices (#473)
* Add mozilla plugin key to manifest

* Move all chrome references into platform-checking module

Addresses #453

* Add chrome global back to linter blacklist

* Add tests
2016-07-21 10:45:32 -07:00