Compatibility with new eslint config spec (flat config)

[jjangga0214]

Eslint announced a new config spec.
https://eslint.org/docs/latest/user-guide/configuring/configuration-files-new

According to eslint's official blog, though backward compatibility was taken care of, not all shareable configs continue to work with the new config system.

Is eslint-config-prettier compatible with the new config?

Thanks.

[lydell]

I tried it out quickly, and in short: Yes, it works. But there are some things to do as well.

eslint.config.js:

import prettier from "eslint-config-prettier";

export default [
    "eslint:recommended",
    prettier,
];

The above successfully disables the no-mixed-spaces-and-tabs rule from eslint:recommended! 🎉

And this:

import prettier from "eslint-config-prettier";

export default [
    {
        rules: {
            semi: "error",
        }
    },
    prettier,
];

The above successfully turns off the semi rule. 🎉 Even though semi wasn’t set by another config. This wasn’t possible before.

However, the CLI helper tool does not work:

❯ npx eslint-config-prettier index.js
No ESLint configuration found in /Users/lydell/stuff/eslint.

TODO:

• Make the CLI helper tool work and make sense with the new config format.
• Update documentation with examples of both old and new format.
• Make it clear in the documentation that you need to put prettier after things that you want to override, and that it can “win” over not just other configs but also your own config (as mentioned above).

However, it might be too early days to do this just yet: eslint/eslint#13481

[lydell]

Changing this line:

eslint-config-prettier/bin/cli.js

Line 11 in a57a3bb

const { ESLint } = require(require.resolve("eslint", {

-const { ESLint } = require(require.resolve("eslint", {
+const { FlatESLint: ESLint } = require(require.resolve("eslint/use-at-your-own-risk", {

… seems to make the CLI work with the new config system. However, that is still an unsupported API, and I need to figure out how to support both the old and new config systems at the same time.

I think the conclusion for now is:

• The main part of eslint-config-prettier works already with the new config system.
• Early adopters can find this issue for information.
• Let’s wait for ESLint to progress more. The above experiment indicates that we should be able to make the CLI work without too much effort when the time comes.

[theoparis]

Unless I'm doing something wrong it does not seem to work anymore 😕

Oops! Something went wrong! :(

ESLint: 8.44.0

TypeError: Cannot read properties of undefined (reading 'includes')
    at /home/theo/dev/fem-kanban/node_modules/.pnpm/eslint-plugin-prettier@5.0.0-alpha.2_eslint-config-prettier@8.8.0_eslint@8.44.0_prettier@3.0.0/node_modules/eslint-plugin-prettier/worker.js:116:27

[lydell]

The updated CLI tool was released in v9.0.0.

The docs and tests have been updated too.