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

no-confusing-arrow rule is already fixed in ESLint 2.4.0 #791

Closed
Ky6uk opened this issue Mar 16, 2016 · 2 comments
Closed

no-confusing-arrow rule is already fixed in ESLint 2.4.0 #791

Ky6uk opened this issue Mar 16, 2016 · 2 comments

Comments

@Ky6uk
Copy link

Ky6uk commented Mar 16, 2016

#752 looks like fixed, right? I think we can enable it back.

@ljharb
Copy link
Collaborator

ljharb commented Mar 16, 2016

Good call - we'd need to enable the allowParens option.

@robbinjanssen
Copy link

For some reason this is still not working for me.

const appIsUnlocked = (resolve, reject) => (req, res, next) =>
  (isUnlocked(req) === true) ? resolve(req, res, next) : reject(req, res, next);

Using:

"eslint": "^2.5.1",
"eslint-config-airbnb": "^6.2.0",
"eslint-plugin-react": "^4.2.3",

I'd be happy to provide you with more info if needed.

Edit
Never mind, had to read the docs a bit better :)
This works:

const appIsUnlocked = (resolve, reject) => (req, res, next) =>
  (isUnlocked(req) === true ? resolve(req, res, next) : reject(req, res, next));

jaylaw81 pushed a commit to appirio-digital/ads-best-practices that referenced this issue Sep 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants