-
Notifications
You must be signed in to change notification settings - Fork 4
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
Migrate to ESM, ESLint 9 and flat config #13
Conversation
4783017
to
d41e118
Compare
d41e118
to
c676346
Compare
4d806c0
to
4ff7efc
Compare
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.
Looks good from my end. One thing to consider: https://github.com/camunda/eslint-plugin-camunda-licensed/pull/13/files#r1791441298.
c76ace3
to
b7d910a
Compare
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.
Two small nit-picks, and potential to include bpmn-io/eslint-plugin-bpmn-io@55c414d.
package.json
Outdated
".": "./index.js", | ||
"./configs/apache.js": "./configs/apache.js", | ||
"./configs/commercial.js": "./configs/commercial.js", | ||
"./configs/mit.js": "./configs/mit.js" |
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.
"./configs/mit.js": "./configs/mit.js" | |
"./configs/mit.js": "./configs/mit.js", | |
"./package.json": "./package.json" |
Related to bpmn-io/internal-docs#1042 BREAKING CHANGE: require eslint@9 (drop support for <= eslint@8) BREAKING CHANGE: use from ESM code-base, this module is ESM only BREAKING CHANGE: consume as ES module
b7d910a
to
9c92528
Compare
Thanks! I'll go ahead and merge + release this. |
Updates the library to support eslint@9 (only), as eslint@8 is out of maintainance, as of October, 1st.
Turns this into an ESM only package, so consume it in
.mjs
files in commonjs projects.Related to https://github.com/bpmn-io/internal-docs/issues/1042
Related to https://github.com/bpmn-io/internal-docs/issues/883