Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More lint tweaking #3164

Merged
merged 1 commit into from
Dec 3, 2016
Merged

Conversation

acinader
Copy link
Contributor

@acinader acinader commented Dec 2, 2016

  1. Add no space in paren rule
  2. fix spec/eslintrc.json so it allow for inheriting from root rc.

Because the spec/eslintrc.json specified recommended, it "turned off" all of the
rule tweaks in the root. This fixes that so that there is no need to duplicate anything (in fact you shouldn't) from the root eslintrc.json.....nice

@flovilmart
Copy link
Contributor

Still on trusty, need to rebase against master or hit update branch

1. Add no space in paren rule
2. fix spec/eslintrc.json so it allow for inheriting from root rc.

Because the spce rc specified reccomended, it "turned off" all of the
rule tweaks in the root.  This fixes that.
@facebook-github-bot
Copy link

@acinader updated the pull request - view changes

@acinader
Copy link
Contributor Author

acinader commented Dec 2, 2016

i just rebase to pick up the travis changes on both open prs of mine...

@flovilmart flovilmart merged commit a270632 into parse-community:master Dec 3, 2016
rsouzas pushed a commit to back4app/parse-server that referenced this pull request Dec 3, 2016
1. Add no space in paren rule
2. fix spec/eslintrc.json so it allow for inheriting from root rc.

Because the spce rc specified reccomended, it "turned off" all of the
rule tweaks in the root.  This fixes that.
Jcarlosjunior pushed a commit to back4app/parse-server that referenced this pull request Dec 13, 2016
1. Add no space in paren rule
2. fix spec/eslintrc.json so it allow for inheriting from root rc.

Because the spce rc specified reccomended, it "turned off" all of the
rule tweaks in the root.  This fixes that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants