Skip to content

Conversation

@erisu
Copy link
Contributor

@erisu erisu commented Oct 30, 2025

Replaced current dev dependencies:

"devDependencies": {
  "eslint": "^8.32.0",
  "eslint-config-semistandard": "^17.0.0",
  "eslint-config-standard": "^17.0.0",
  "eslint-plugin-import": "^2.27.5",
  "eslint-plugin-jest": "^27.2.1",
  "eslint-plugin-node": "^11.1.0",
  "eslint-plugin-promise": "^6.1.1"
},

with:

"devDependencies": {
  "@cordova/eslint-config": "^6.0.0"
},

The @cordova/eslint-config package is largely identical to what we need in this package.

The main difference between using @cordova/eslint-config and requiring the packages directly is that the standard and semistandard rules have been manually included in @cordova/eslint-config, removing the need to install them as dependencies. In the past, these dependencies were slow to update and often lagged behind newer ESLint releases. Because this delay hindered our ability to upgrade ESLint in a timely manner, the rules were integrated directly into the package.

While we could make similar changes in this repository to continue including the packages directly, I felt that using @cordova/eslint-config would be faster to support since it already includes most of the required changes. Additionally, @cordova/eslint-config uses eslint-plugin-n instead of eslint-plugin-node, which is no longer maintained. It does not come with eslint-plugin-jest, which we can drop anyway since we no longer use Jest.

Release Notes for @cordova/eslint-config@6.0.0

@erisu erisu requested a review from dpogue October 30, 2025 03:09
@erisu erisu added this to the 3.0.0 milestone Oct 30, 2025
@dpogue dpogue merged commit 5a6aab3 into ios-control:master Oct 30, 2025
4 checks passed
@dpogue dpogue mentioned this pull request Oct 30, 2025
6 tasks
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.

2 participants