Chore: enable quote avoidEscape option in eslint-config-eslint

[aladdin-add]

it can be more readable to use:

var s = `"hello world!"`;

than

var s = "\"hello world!\"";

hopefully, the option can be enabled by default?

[aladdin-add]

still not working, but that seems a bug to me. (#10627)

[platinumazure]

Hi @aladdin-add, thanks for the PR.

Just out of curiosity, what problem are you trying to solve here? Is the current configuration actually causing a lot of problems (i.e., are there many strings in ESLint or another repository which suffer from lots of escape sequences)?

[aladdin-add]

the problem is the current config disallows a better code(at least in my view).

a common usage is code tests:

valid: [
  "var s = \"hello\";",
]

[aladdin-add]

I'm seeing a large number of cases using escape, seems enabling the option could be easier to write and read. thoughts?
/cc @platinumazure @eslint/eslint-team

e.g.

eslint/tests/lib/rules/no-duplicate-imports.js

Lines 23 to 47 in cb946af

	"import os from \"os\";\nimport fs from \"fs\";",
	"import { merge } from \"lodash-es\";",
	"import _, { merge } from \"lodash-es\";",
	"import * as Foobar from \"async\";",
	"import \"foo\"",
	"import os from \"os\";\nexport { something } from \"os\";",
	{
	code: "import os from \"os\";\nexport { hello } from \"hello\";",
	options: [{ includeExports: true }]
	},
	{
	code: "import os from \"os\";\nexport * from \"hello\";",
	options: [{ includeExports: true }]
	},
	{
	code: "import os from \"os\";\nexport { hello as hi } from \"hello\";",
	options: [{ includeExports: true }]
	},
	{
	code: "import os from \"os\";\nexport default function(){};",
	options: [{ includeExports: true }]
	},
	{
	code: "import { merge } from \"lodash-es\";\nexport { merge as lodashMerge }",
	options: [{ includeExports: true }]

[platinumazure]

Thanks @aladdin-add. I'm 👍 to this change if it makes our rule tests easier to read.

[platinumazure]

@eslint/eslint-team Any thoughts on this? Seems reasonable to make some of our tests easier to write (IMO).

[aladdin-add]

thanks for the feedback, seems we have more than 4 👍 from the team members. labelled accepted.