Eslint imports #3241

Merged
merged 3 commits into from Oct 25, 2016

Projects

None yet

4 participants

@RomanovRoman
Contributor

Proposed changes

  • Add "npm run esfix" command
  • Add "no-multi-spaces" linting rule
  • Fix new linting errors

Link to the issue: #3166

RomanovRoman added some commits Oct 22, 2016
@RomanovRoman RomanovRoman Add "npm run esfix" command 121d047
@RomanovRoman RomanovRoman Add "no-multi-spaces" linting rule 8efb544
@RomanovRoman RomanovRoman Fix linting errors with "npm run esfix"
e887d16
@coveralls

Coverage Status

Coverage remained the same at 100.0% when pulling e887d16 on RomanovRoman:eslint_imports into 7f31924 on marionettejs:master.

@scott-w

This looks good to me. Thanks for your work @RomanovRoman!

@paulfalgout paulfalgout merged commit c72e95a into marionettejs:master Oct 25, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment