Skip to content

eslint v4 support #1447

Closed
microsoftgraph/nodejs-webhooks-sample
#10

Description

@ljharb

Note: I filed this issue within minutes of eslint v4.0.0 being released.

eslint-config-airbnb and eslint-config-airbnb-base do not yet support eslint v4.

If it is easy/possible to support both v3 and v4, we will release a non-major bump of both packages that does so, before releasing a major bump that only supports v4.

This will likely take a few weeks at a minimum.

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

Peer Dep eslint combined v3 and v4 support released:

  • eslint-plugin-import issue | PR | released as v2.6.0
  • eslint-plugin-react issue for v7.1.0 release
  • eslint-plugin-jsx-a11y issue | PR | release (v5.1.0 and v6)
  • eslint-find-rules issue | released as v30.1
  • update and publish the base config
  • update and publish the main config

Activity

  1. self-assigned this
    on Jun 12, 2017
  2. deleted a comment from willin on Jun 17, 2017
  3. added a commit that references this issue on Jun 20, 2017
  4. Geczy commented on Jun 28, 2017

    @Geczy

    Any updates to this?

  5. ljharb commented on Jun 28, 2017

    @ljharb
    CollaboratorAuthor

    @Geczy I've kept the OP updated; if there's updates, you'd see them here.

    Since there are no updates posted, no, there are no updates. Please be patient - the OP said "a few weeks at minimum" and it's not yet been 3.

  6. MFry commented on Jul 6, 2017

    @MFry

    It would be useful to post somewhere prominent on what would be recommended for new projects and the best way to stay on the 3x branch for eslint.

    yarn add eslint@3 for yarn users would go a long way to avoid setup hassles.

  7. EloB commented on Jul 7, 2017

    @EloB

    Really would like this feature. The ability to use overrides would be awesome!

    "overrides": [
      {
        "files": ["src/**/__tests__/*.js"],
        "env": {
          "jest": true
        }
      }
    ]
    
  8. added a commit that references this issue on Jul 7, 2017
  9. ljharb commented on Jul 9, 2017

    @ljharb
    CollaboratorAuthor

    @MFry i believe yarn has --peer, such that yarn add eslint-config-airbnb --peer should just work? If not, I'd file a bug on yarn. Note that the existing instructions have no setup hassles with npm.

    @EloB i'd strongly discourage you from colocating tests; but I agree that if you're doing so, it's a useful feature to be able to configure eslint with globs.

  10. travi commented on Jul 9, 2017

    @travi

    @ljharb would you be willing to expand on the reason for discouraging test colocation?

    i used to be more on the side of separating, and still do at this point. however, i've mostly come around to the idea of keeping them with the source files and eslint is the last thing keeping me from doing so.

    i'm interested in your specific thoughts if you'd be willing to share.

  11. ljharb commented on Jul 9, 2017

    @ljharb
    CollaboratorAuthor

    @travi that's really a topic for another thread (feel free to file one); and I have a litany of reasons. However, most of those are moot because "not all tools support glob-based configuration". eslint only added it in v4, babel doesn't, etc.

  12. Geczy commented on Jul 10, 2017

    @Geczy

    No updates since 2 weeks ago, it's now been 1 month, any news? Do you guys need help taking this on from contributors?

  13. MFry commented on Jul 10, 2017

    @MFry

    @ljharb npm3 removed easy install of peer dependencies and the support is not yet available in yarn.
    Doing yarn add eslint-config-airbnb --peer would just install the package in your peerdependencies. :(

    From NPM:

    #6930 (#6565) peerDependencies no longer cause anything to be implicitly installed. Instead, npm will now warn if a packages peerDependencies are missing, but it's up to the consumer of the module (i.e. you) to ensure the peers get installed / are included in package.json as direct dependencies or devDependencies of your package.
    
  14. ljharb commented on Jul 11, 2017

    @ljharb
    CollaboratorAuthor

    @MFry ah, i misunderstood what --peer does for yarn. In that case, I don't have an easy solution to install the needed peer deps with yarn - but since npm 5 exists, I'm not sure why you'd need yarn :-)

    I'm aware that npm 3+ no longer automatically installs peer deps; that's why the readme of both of our eslint config packages has instructions to do so.

    @Geczy no, no help is needed. There's no urgency; and the update will come when it's ready.

  15. 21 remaining items

  16. mark007 commented on Jul 24, 2017

    @mark007

    @ljharb Hi, this error still occurs when extending from airbnb/legacy. Is this the intention or should it also be updated to remove the top level ecmaFeatures?

  17. shaharmor commented on Jul 24, 2017

    @shaharmor

    @ljharb airbnb/legacy has some issues with v4 still

  18. ljharb commented on Jul 24, 2017

    @ljharb
    CollaboratorAuthor

    Thanks, legacy is fixed in 0d938ae and will be released in eslint-config-airbnb-base shortly.

  19. wkurniawan07 commented on Aug 13, 2017

    @wkurniawan07

    Hi, any updates on when the ESLint v4-exclusive versions (v12.0.0 base and v16.0.0 main) will be released? Thanks for all the awesome works!

  20. ljharb commented on Aug 13, 2017

    @ljharb
    CollaboratorAuthor

    No updates.

  21. mark007 commented on Aug 29, 2017

    @mark007

    @ljharb is there any version of eslint-config-airbnb that includes the fix for the legacy.js file when introducing eslint v4, ie this patch 0d938ae

    I don't think its included in the latest 15.1.0 of eslint-config-airbnb that I can see?

  22. ljharb commented on Aug 29, 2017

    @ljharb
    CollaboratorAuthor

    It's automatically included because of the semver range on eslint-config-airbnb-base.

  23. ljharb commented on Oct 6, 2017

    @ljharb
    CollaboratorAuthor

    v16 is now released.

  24. added a commit that references this issue on Feb 5, 2024
    16c73d0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions