Skip to content
This repository has been archived by the owner on May 31, 2021. It is now read-only.

Expand eslint rules #555

Closed
jfurrow opened this issue Nov 28, 2017 · 4 comments
Closed

Expand eslint rules #555

jfurrow opened this issue Nov 28, 2017 · 4 comments

Comments

@jfurrow
Copy link
Member

jfurrow commented Nov 28, 2017

Let's have better eslint rules.

I think I'd like to use eslint-config-airbnb for the client-side JS and eslint-config-airbnb-base for server-side.

Any thoughts?

@jfurrow jfurrow added the code label Nov 28, 2017
@dcousens
Copy link
Contributor

Personally I use https://standardjs.com/ - that will be my +1 👍

@noraj
Copy link
Contributor

noraj commented Nov 28, 2017

I'm for the more standard lint rules and standardjs seems to be good for that. Additionally it seems to be supported by some atom plugins like linter-js-standard.

Update: I don't know much of javascript so I don't know the advantage of ESLint but the standardjs can be supported by ESLint with those plugins eslint-config-standard or eslint-config-standard-react.

@noraj
Copy link
Contributor

noraj commented May 2, 2018

I created the clean/lint branch where I began to clean the code after doing a npm run lint.

I'm 100% sure for Unnecessary escape character and remove Duplicate key but not for all remove unused value and maybe the strict equality.

@jesec
Copy link
Member

jesec commented Jan 31, 2021

Flood now uses airbnb-typescript with minimal exceptions.

@jesec jesec closed this as completed Jan 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants