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
Bug: npm init @eslint/config сreates a non-working installation #93
Comments
Hi, I have the same proglem. Do you know how to fix it. And something more. eslint creates .mjs file for me. and it cannot be even detected. i change the suffix to .js and then have this error. |
seems as of now, you can use: $ npm init @eslint/config -- --eslintrc --config eslint-config-standard-with-typescript |
@aladdin-add ok, now my config looklike: const compat = new FlatCompat({baseDirectory: __dirname, recommendedConfig: pluginJs.configs.recommended});
export default [
...compat.extends("standard-with-typescript"),
]; How to customize these rules? |
In fact, I had to add another “files” field: export default [
{
files: [ 'src/**.tsx' ]
},
...compat.extends("standard-with-typescript"),
]; and run the linter as follows: Otherwise he didn't check anything... |
export default [
...(compat.extends("standard-with-typescript").map(config => ({files: ["**/*.tsx"], ...config}))),
]; |
Ok, seems like it work for me, thank's |
This is installing eslint ^8.0.1 - is this intentional?
|
This is the strategy of the package manager: some of these dependencies require eslint v8, and do not support eslint v9 yet. |
and seems the package has been deprecated, related #95 |
Environment
Node version: v18.19.1
npm version: v10.5.1
Local ESLint version: 8.57.0
Global ESLint version: --
Operating System: Ubuntu 23.10
What parser are you using?
Default (Espree)
What did you do?
First of all, I want to note that for some reason
npm init @eslint/config
is installing version 8 of ESLint, not 9, although my version of the Node is supported, according to your guide ("Node.js < v18.18, v19 are no longer supported").But that’s okay (i really don’t care what to use, 8th or 9th, linter of any version is ****, but unfortunately this is my company’s choice). What's worse is that config command creates a non-working eslint.config.js:
Configuration
If we try to run
npx lint
we get the following error:Error: Key "plugins": Cannot redefine plugin "@typescript-eslint".
Okay it's look like tseslint.configs.recommended is redefine some fields of compat.extends("standard-with-typescript")... But if we just comment one of these rules, still nothing works, the linter just exits without messages.
What did you expect to happen?
I would like the linter to work after the config, and the guides to have correct, up-to-date information
What actually happened?
Linter doesn't work, and the guides are a mess
Link to Minimal Reproducible Example
https://eslint.org/play/#eyJ0ZXh0IjoiaW1wb3J0IGdsb2JhbHMgZnJvbSBcImdsb2JhbHNcIjtcbmltcG9ydCB0c2VzbGludCBmcm9tIFwidHlwZXNjcmlwdC1lc2xpbnRcIjtcbmltcG9ydCBwbHVnaW5SZWFjdENvbmZpZyBmcm9tIFwiZXNsaW50LXBsdWdpbi1yZWFjdC9jb25maWdzL3JlY29tbWVuZGVkLmpzXCI7XG5cbmltcG9ydCBwYXRoIGZyb20gXCJwYXRoXCI7XG5pbXBvcnQgeyBmaWxlVVJMVG9QYXRoIH0gZnJvbSBcInVybFwiO1xuaW1wb3J0IHsgRmxhdENvbXBhdCB9IGZyb20gXCJAZXNsaW50L2VzbGludHJjXCI7XG5pbXBvcnQgcGx1Z2luSnMgZnJvbSBcIkBlc2xpbnQvanNcIjtcblxuLy8gbWltaWMgQ29tbW9uSlMgdmFyaWFibGVzIC0tIG5vdCBuZWVkZWQgaWYgdXNpbmcgQ29tbW9uSlNcbmNvbnN0IF9fZmlsZW5hbWUgPSBmaWxlVVJMVG9QYXRoKGltcG9ydC5tZXRhLnVybCk7XG5jb25zdCBfX2Rpcm5hbWUgPSBwYXRoLmRpcm5hbWUoX19maWxlbmFtZSk7XG5jb25zdCBjb21wYXQgPSBuZXcgRmxhdENvbXBhdCh7YmFzZURpcmVjdG9yeTogX19kaXJuYW1lLCByZWNvbW1lbmRlZENvbmZpZzogcGx1Z2luSnMuY29uZmlncy5yZWNvbW1lbmRlZH0pO1xuXG5leHBvcnQgZGVmYXVsdCBbXG4gIHtsYW5ndWFnZU9wdGlvbnM6IHsgZ2xvYmFsczogZ2xvYmFscy5icm93c2VyIH19LFxuICAuLi5jb21wYXQuZXh0ZW5kcyhcInN0YW5kYXJkLXdpdGgtdHlwZXNjcmlwdFwiKSxcbiAgLi4udHNlc2xpbnQuY29uZmlncy5yZWNvbW1lbmRlZCxcbiAgcGx1Z2luUmVhY3RDb25maWcsXG5dOyIsIm9wdGlvbnMiOnsicnVsZXMiOnsiY29uc3RydWN0b3Itc3VwZXIiOlsiZXJyb3IiXSwiZm9yLWRpcmVjdGlvbiI6WyJlcnJvciJdLCJnZXR0ZXItcmV0dXJuIjpbImVycm9yIl0sIm5vLWFzeW5jLXByb21pc2UtZXhlY3V0b3IiOlsiZXJyb3IiXSwibm8tY2FzZS1kZWNsYXJhdGlvbnMiOlsiZXJyb3IiXSwibm8tY2xhc3MtYXNzaWduIjpbImVycm9yIl0sIm5vLWNvbXBhcmUtbmVnLXplcm8iOlsiZXJyb3IiXSwibm8tY29uZC1hc3NpZ24iOlsiZXJyb3IiXSwibm8tY29uc3QtYXNzaWduIjpbImVycm9yIl0sIm5vLWNvbnN0YW50LWJpbmFyeS1leHByZXNzaW9uIjpbImVycm9yIl0sIm5vLWNvbnN0YW50LWNvbmRpdGlvbiI6WyJlcnJvciJdLCJuby1jb250cm9sLXJlZ2V4IjpbImVycm9yIl0sIm5vLWRlYnVnZ2VyIjpbImVycm9yIl0sIm5vLWRlbGV0ZS12YXIiOlsiZXJyb3IiXSwibm8tZHVwZS1hcmdzIjpbImVycm9yIl0sIm5vLWR1cGUtY2xhc3MtbWVtYmVycyI6WyJlcnJvciJdLCJuby1kdXBlLWVsc2UtaWYiOlsiZXJyb3IiXSwibm8tZHVwZS1rZXlzIjpbImVycm9yIl0sIm5vLWR1cGxpY2F0ZS1jYXNlIjpbImVycm9yIl0sIm5vLWVtcHR5IjpbImVycm9yIl0sIm5vLWVtcHR5LWNoYXJhY3Rlci1jbGFzcyI6WyJlcnJvciJdLCJuby1lbXB0eS1wYXR0ZXJuIjpbImVycm9yIl0sIm5vLWVtcHR5LXN0YXRpYy1ibG9jayI6WyJlcnJvciJdLCJuby1leC1hc3NpZ24iOlsiZXJyb3IiXSwibm8tZXh0cmEtYm9vbGVhbi1jYXN0IjpbImVycm9yIl0sIm5vLWZhbGx0aHJvdWdoIjpbImVycm9yIl0sIm5vLWZ1bmMtYXNzaWduIjpbImVycm9yIl0sIm5vLWdsb2JhbC1hc3NpZ24iOlsiZXJyb3IiXSwibm8taW1wb3J0LWFzc2lnbiI6WyJlcnJvciJdLCJuby1pbnZhbGlkLXJlZ2V4cCI6WyJlcnJvciJdLCJuby1pcnJlZ3VsYXItd2hpdGVzcGFjZSI6WyJlcnJvciJdLCJuby1sb3NzLW9mLXByZWNpc2lvbiI6WyJlcnJvciJdLCJuby1taXNsZWFkaW5nLWNoYXJhY3Rlci1jbGFzcyI6WyJlcnJvciJdLCJuby1uZXctbmF0aXZlLW5vbmNvbnN0cnVjdG9yIjpbImVycm9yIl0sIm5vLW5vbm9jdGFsLWRlY2ltYWwtZXNjYXBlIjpbImVycm9yIl0sIm5vLW9iai1jYWxscyI6WyJlcnJvciJdLCJuby1vY3RhbCI6WyJlcnJvciJdLCJuby1wcm90b3R5cGUtYnVpbHRpbnMiOlsiZXJyb3IiXSwibm8tcmVkZWNsYXJlIjpbImVycm9yIl0sIm5vLXJlZ2V4LXNwYWNlcyI6WyJlcnJvciJdLCJuby1zZWxmLWFzc2lnbiI6WyJlcnJvciJdLCJuby1zZXR0ZXItcmV0dXJuIjpbImVycm9yIl0sIm5vLXNoYWRvdy1yZXN0cmljdGVkLW5hbWVzIjpbImVycm9yIl0sIm5vLXNwYXJzZS1hcnJheXMiOlsiZXJyb3IiXSwibm8tdGhpcy1iZWZvcmUtc3VwZXIiOlsiZXJyb3IiXSwibm8tdW5kZWYiOlsiZXJyb3IiXSwibm8tdW5leHBlY3RlZC1tdWx0aWxpbmUiOlsiZXJyb3IiXSwibm8tdW5yZWFjaGFibGUiOlsiZXJyb3IiXSwibm8tdW5zYWZlLWZpbmFsbHkiOlsiZXJyb3IiXSwibm8tdW5zYWZlLW5lZ2F0aW9uIjpbImVycm9yIl0sIm5vLXVuc2FmZS1vcHRpb25hbC1jaGFpbmluZyI6WyJlcnJvciJdLCJuby11bnVzZWQtbGFiZWxzIjpbImVycm9yIl0sIm5vLXVudXNlZC1wcml2YXRlLWNsYXNzLW1lbWJlcnMiOlsiZXJyb3IiXSwibm8tdW51c2VkLXZhcnMiOlsiZXJyb3IiXSwibm8tdXNlbGVzcy1iYWNrcmVmZXJlbmNlIjpbImVycm9yIl0sIm5vLXVzZWxlc3MtY2F0Y2giOlsiZXJyb3IiXSwibm8tdXNlbGVzcy1lc2NhcGUiOlsiZXJyb3IiXSwibm8td2l0aCI6WyJlcnJvciJdLCJyZXF1aXJlLXlpZWxkIjpbImVycm9yIl0sInVzZS1pc25hbiI6WyJlcnJvciJdLCJ2YWxpZC10eXBlb2YiOlsiZXJyb3IiXX0sImxhbmd1YWdlT3B0aW9ucyI6eyJwYXJzZXJPcHRpb25zIjp7ImVjbWFGZWF0dXJlcyI6e319fX19
Participation
Additional comments
Why is there no detail in the guide on how to configure the config after
npm init @eslint/config
? User's have to figure it out for themselfs?Well, why it was need to update the version until everything was tested? Are you in a hurry somewhere?
The text was updated successfully, but these errors were encountered: