erisu commented on PR #47:
URL: https://github.com/apache/cordova-eslint/pull/47#issuecomment-3082973619
So far, I haven't had any luck with this draft PR.
I tried installing this branch into the `cordova-android` repo for testing.
Since ESLint has changed how configuration files work, I used their migration
tool to update ours.
```zsh
% npx @eslint/migrate-config .eslintrc.yml
Migrating .eslintrc.yml
Also importing your .eslintignore file
Wrote new config to ./eslint.config.mjs
You will need to install the following packages to use the new config:
- @eslint/js
- @eslint/eslintrc
You can install them using the following command:
npm install @eslint/js @eslint/eslintrc -D
```
It suggests installing `@eslint/js` and `@eslint/eslintrc`, which I assume
we'd want to add to this repo. But for testing purposes, I installed them in
the `cordova-android` package.
When I tried to run the linter, I got the following error:
```zsh
% npm run lint
> cordova-android@15.0.0-dev lint
> eslint lib spec test "templates/cordova/**/!(*.*)"
Oops! Something went wrong! :(
ESLint: 9.31.0
TypeError: Converting circular structure to JSON
--> starting at object with constructor 'Object'
| property 'configs' -> object with constructor 'Object'
| property 'flat/recommended-module' -> object with constructor
'Object'
| property 'plugins' -> object with constructor 'Object'
--- property 'n' closes the circle
Referenced from:...
```
--
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: issues-unsubscr...@cordova.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
