-
-
Notifications
You must be signed in to change notification settings - Fork 563
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: allow eslint-plugin-promise v5 as peer dependency #186
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Closing per #183 (comment) |
I prefer this PR @divlo since, as I've argued before, this is the minimal change for fixing the problem of eg. @erezrokah. #183 contains additional changes that are not needed for this very fix and which are possible to object to. |
This happened to be at the top of my inbox so I looked at the small change and approved it 😅 Unfortunately I don't have time to read thru #183 right now, but my take in just quickly glancing at the changes (sorry if this has been discussed) is that it's always easier to merge smaller PRs |
Allow
eslint-plugin-promise
v5 as peer dependency.Fixes the following use case netlify/eslint-config-node#160 (comment) and extracted from #183 to separate concerns.