Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Update lint style for Standard@9.0.0 - 2nd patch #7785

Merged
merged 1 commit into from
Mar 21, 2017
Merged

Update lint style for Standard@9.0.0 - 2nd patch #7785

merged 1 commit into from
Mar 21, 2017

Conversation

cezaraugusto
Copy link
Contributor

@cezaraugusto cezaraugusto commented Mar 18, 2017

Auditors: @bsclifton, @bbondy

Fix #7784
Follow-up of #7495

This PR addressed no-mixed-operators rule

Test plan (two steps):

1. With current standard, run

npm run lint
  • There should be no linting errors

2. Install standard@9.0.0 globally

npm i -g standard@9.0.0

Checkout this PR, under browser-laptop folder, run

standard
  • Lint errors should not include:
    • Unexpected mix of ...

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look great! Merging now so we can avoid conflicts needing rebase 😄

@bsclifton bsclifton merged commit 26a580b into brave:master Mar 21, 2017
@cezaraugusto cezaraugusto deleted the update/standard9.0.0-2st-patch branch July 25, 2017 07:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants