jdaugherty commented on PR #139: URL: https://github.com/apache/grails-forge-ui/pull/139#issuecomment-5013925068
This upgrade is currently **blocked by two plugins that don't yet support eslint 10**, so it can't be merged as-is: 1. `[email protected]` (latest) peers `eslint` only up to `^9`. There is a maintained fork, `eslint-plugin-import-x` (supports eslint 8/9/10), that this config could migrate to. 2. `[email protected]` (latest) peers `eslint` only up to `^9.7`, and **has no eslint-10-compatible release yet**. Forcing it onto eslint 10 crashes at lint time: ``` TypeError: Error while loading rule 'react/display-name': contextOrFilename.getFilename is not a function ``` (eslint 10 removed the legacy rule-context API this plugin version relies on.) `[email protected]` already supports eslint 10, so that one is fine. **Recommendation:** hold at eslint 9 until `eslint-plugin-react` ships an eslint-10-compatible release. eslint is a devDependency (lint-only), so there's no runtime/security exposure to staying on 9 for now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
