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

feat: support ESLint 8 #98

Merged
merged 5 commits into from
Aug 17, 2022
Merged

feat: support ESLint 8 #98

merged 5 commits into from
Aug 17, 2022

Conversation

DiegoRBaquero
Copy link
Contributor

@DiegoRBaquero DiegoRBaquero commented May 10, 2022

This PR updates eslint-plugin-jsdoc and eslint-plugin-promise specifically, which are the only blockers for ESLint 8 adoption. Also fixes repository url.

These changes seems to be non-breaking, i.e. still fully supported on ESLint 7

This PR updates eslint-plugin-jsdoc and eslint-plugin-promise specifically, which are the only blockers for ESLint 8 adoption. Also fixes repository url
@achingbrain
Copy link
Member

It seems there are some incompatible peer dependencies introduced by this PR - @DiegoRBaquero could you take a look please?

@DiegoRBaquero
Copy link
Contributor Author

Taking a look, blocked by mightyiam/eslint-config-love#792

@BigLep BigLep added need/author-input Needs input from the original author P3 Low: Not priority right now labels May 27, 2022
@BigLep
Copy link

BigLep commented May 27, 2022

2022-05-27: we'll check back in on this in a week or two. If mightyiam/eslint-config-love#792 isn't resolved, we'll ask the author to unblock themselves by publishing a fork.

@BigLep
Copy link

BigLep commented Jun 10, 2022

2022-06-10: we will continue to let this sit given the active work in mightyiam/eslint-config-love#792 (comment)

@lidel lidel added need/author-input Needs input from the original author need/maintainers-input Needs input from the current maintainer(s) status/blocked Unable to be worked further until needs are met and removed need/author-input Needs input from the original author labels Jun 24, 2022
@BigLep
Copy link

BigLep commented Jul 22, 2022

@DiegoRBaquero : given mightyiam/eslint-config-love#792 has been resolved, do we need to do any updates or just trigger the build again?

@achingbrain
Copy link
Member

Yes, the eslint-config-standard-with-typescript dep needs updating to ^22.0.0 and the package-lock.json file needs regenerating.

test/validate.js Outdated Show resolved Hide resolved
@achingbrain achingbrain merged commit 511821d into ipfs:default Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/author-input Needs input from the original author need/maintainers-input Needs input from the current maintainer(s) P3 Low: Not priority right now status/blocked Unable to be worked further until needs are met
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants