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

Fix peerDependencies in eslint-config #1953

Merged
merged 6 commits into from
Oct 29, 2020
Merged

Conversation

stephenwade
Copy link
Contributor

@stephenwade stephenwade commented Oct 27, 2020

eslint-config should depend on @babel/eslint-plugin, not the legacy eslint-plugin-babel. This fixes install errors when using npm@7, which now automatically installs peer dependencies.

Closes #1951.

@changeset-bot
Copy link

changeset-bot bot commented Oct 27, 2020

🦋 Changeset detected

Latest commit: b485406

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@open-wc/eslint-config Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

I'm not sure what I did wrong when adding this changeset. I'm deleting this file and going to let the maintainers add a changeset properly.
Copy link

@felixapitzsch felixapitzsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This very simple one-line change fixes the issue

@felixapitzsch
Copy link

Could someone from this project (with write access) please review and approve this PR? It is a really simple fix. However, the issue is really blocking me and my colleagues. And all other means to work around the issue are very hacky.

@thepassle
Copy link
Member

Hrm. Looks like prettier fails on the auto generated changeset. Thats something we'll have to look into and fix, for now, could you pull and run yarn format:prettier from the root of the project, and commit the changes to make the CI pass?

@daKmoR
Copy link
Member

daKmoR commented Oct 29, 2020

formatted it 👍

thank you 🤗

@daKmoR daKmoR merged commit 87ce50c into open-wc:master Oct 29, 2020
@stephenwade stephenwade deleted the patch-1 branch October 29, 2020 19:46
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.

eslint-config: wrong peer dependency eslint-plugin-babel@^7.6.0
4 participants