Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

[ESLint] autofix style and minor issues #1379

Merged
merged 5 commits into from
Nov 4, 2016
Merged

Conversation

luclu
Copy link
Contributor

@luclu luclu commented Nov 3, 2016

Applies following fixes among others across the code-base:

  • intend
  • max. 2 empty lines
  • spaces after keywords
  • use const/let instead of var (in the appropriate context)
  • prefer arrow callbacks

Basically applies all fixable rules (wrench-icon) enabled in airbnb-base config.

Travis and local builds / dev runs tested.

@mention-bot
Copy link

@luclu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @alexvandesande, @evertonfraga and @hiddentao to be potential reviewers.

@luclu luclu changed the title Luclu eslint autofix [ESLint] autofix style, minor issues Nov 3, 2016
@luclu luclu changed the title [ESLint] autofix style, minor issues [ESLint] autofix style and minor issues Nov 4, 2016
@luclu luclu merged commit 8d952c7 into develop Nov 4, 2016
@frozeman frozeman deleted the luclu_eslint-autofix branch November 7, 2016 11:22
@evertonfraga evertonfraga modified the milestone: 0.8.8 Nov 16, 2016
@lock
Copy link

lock bot commented Mar 31, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants