eslint v5 support

[ljharb]

Just like #1447, I'm filing this in advance.

v17.0.0 of eslint-config-airbnb and v13.0.0 of eslint-config-airbnb-base have just been released.

eslint v5 is planned to be released in the next 24-48 hours, and the plan for our configs will be to either support v3 || v4 || v5, or, if necessary, to drop support for v3 and/or v4 as part of this release.

The timeframe is "probably not as soon as you would like, but as soon as we can".
Please accept this, and please add no noise to this thread beyond emoji reactions - distractions won't help anyone get the work done any faster.

Please do not file additional issues and do not file pull requests attempting to add support - please, instead, continue to use eslint v4 until such time as our configs support v5.

Peer Dep eslint combined v3 and v4 and v5 support released, or v5 support released (depending on which path we need to take):

• eslint-plugin-import
  • PR
  • Released as v2.13.0
• eslint-find-rules
  • PR
  • Release as v3.3.1
• update the base config
• publish the base config (v13.1.0)
• eslint-plugin-react
  • PR
  • Released as v7.10.0
• eslint-plugin-jsx-a11y
  • PR
  • Released as v6.1.0
• eslint-plugin-react
  • enable new option in (jsx-one-expression-per-line](react/jsx-one-expression-per-line jsx-eslint/eslint-plugin-react#1775)
  • released as v7.11.0
  • updated in the main config
• a11y plugin: fix idlist bug (PR) and release
• a11y plugin: label-has-for bug
• update and publish the main config

[YasharF]

Thank you for the great work. It looks like the remaining tasks on the list are the publishing tasks. Do you have any estimates on when to expect the new version to be published?

[ljharb]

I'm waiting on a few bugs (that i've just added).

[bjankord]

Hey @ljharb, curious what that status is on this. I saw some bugfixes landed in jsx-a11y-plugin.

[danactive]

I would love to read why "react/jsx-one-expression-per-line" is enabled. Is there a trend for developers to rotate their monitors vertically after this change 😆

This changes one readable line

<App>Hello World</App>

to an uncomfortable syntax

<App>
  { 'Hello' }
  { ' ' }
  { 'World' }
</App>

[ryansully]

@danactive That rule was turned off in b6a268f.
Hopefully the next release that includes the commit above will be out soon.

Your concern was also brought up here and here.

[ljharb]

eslint-config-airbnb v17.1.0 and eslint-config-airbnb-base v13.1.0 have been released, with eslint 5 support.

[tunnckoCore]

Yeah!

[kmaitski]

Is anyone still having the same issue after upgrading?

Here are my devDependencies, but I am still running into the same issue.

`
"eslint": "^5.3.0",

    "eslint-config-airbnb": "^17.1.0",

    "eslint-config-airbnb-base": "^13.1.0",

    "eslint-loader": "^2.1.0",

    "eslint-plugin-import": "^2.14.0",

    "eslint-plugin-jsx-a11y": "^6.1.1",

    "eslint-plugin-react": "^7.10.0",

`

Am I doing something wrong?

[ljharb]

@kmaitski what does npm ls say? Have you tried running the peer deps install command in the readme for eslint-config-airbnb?

Either way, please file a new issue if you're still having trouble.