feat: add regex option in no-restricted-imports

[snitin315]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[x] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

Fix #18536

/*eslint no-restricted-imports: ["error", { patterns: [{
    "importNamePattern": "_Enum$",
    "regexGroup": "@app/(?!(api/enums$)).*"
}]}]*/

// error
import { Foo_Enum } from '@app/api';
import { Bar_Enum } from '@app/api/bar';
import { Baz_Enum } from '@app/api/baz';
import { B_Enum } from '@app/api/enums/foo';

// no error
import { C_Enum } from '@app/api/enums';

Is there anything you'd like reviewers to focus on?

Earlier group was a required property in the schema and would throw an error if not defined, which isn't the case anymore as we need either group or regexGroup.

[netlify]

✅ Deploy Preview for docs-eslint ready!

Name	Link
🔨 Latest commit	0f8ef13
🔍 Latest deploy log	https://app.netlify.com/sites/docs-eslint/deploys/668126fbc5e2f2000817328f
😎 Deploy Preview	https://deploy-preview-18622--docs-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[mdjermanovic]

Suggested change

{ required: ["group", "regex"] },

I think this restriction is now already covered by the oneOf below.

[snitin315]

yes, updated.

[aladdin-add]

LGTM. Would like @mdjermanovic to verify before merging.

[mdjermanovic]

I think the documentation for patterns needs more updates because, throughout the ### patterns section, it was assumed that it's always based on .gitignore-style patterns and that group property must be specified.

For example:

eslint/docs/src/rules/no-restricted-imports.md

Line 287 in 69ff64e

This is also an object option whose value is an array. This option allows you to specify multiple modules to restrict using `gitignore`-style patterns.

eslint/docs/src/rules/no-restricted-imports.md

Line 314 in 69ff64e

Because the patterns follow the `gitignore`-style, if you want to reinclude any particular module this can be done by prefixing a negation (`!`) mark in front of the pattern. (Negated patterns should come last in the array because order is important.)

eslint/docs/src/rules/no-restricted-imports.md

Line 438 in 69ff64e

This is a boolean option and sets the patterns specified in the `group` array to be case-sensitive when `true`. Default is `false`.

eslint/docs/src/rules/no-restricted-imports.md

Line 481 in 69ff64e

You can also specify `importNames` on objects inside of `patterns`. In this case, the specified names are applied only to the specified `group`.

eslint/docs/src/rules/no-restricted-imports.md

Line 527 in 69ff64e

You can also specify `allowImportNames` on objects inside of `patterns`. In this case, the specified names are applied only to the specified `group`.

[snitin315]

@mdjermanovic Updated.

[mdjermanovic]

LGTM, thanks!