Update: add allowSeparatedGroups option to sort-imports (fixes #12951)

[mdjermanovic]

Prerequisites checklist

• I have read the contributing guidelines.

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

[X] Changes an existing rule

fixes #12951

What changes did you make? (Give an overview)

Added allowSeparatedGroups option to the sort-imports rule.

Examples of correct code for this rule with the { "allowSeparatedGroups": true } option:

/*eslint sort-imports: ["error", { "allowSeparatedGroups": true }]*/

import b from 'foo.js';
import c from 'bar.js';

import a from 'baz.js';

/*eslint sort-imports: ["error", { "allowSeparatedGroups": true }]*/

import b from 'foo.js';
import c from 'bar.js';
// comment
import a from 'baz.js';

/*eslint sort-imports: ["error", { "allowSeparatedGroups": true }]*/

import b from 'foo.js';
import c from 'bar.js';
quux();
import a from 'baz.js';

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

[nzakas]

LGTM. Thanks!

[joshuapinter]

So funny. Was trying to set this option and got the following error:

Error: .eslintrc.json:
	Configuration for rule "sort-imports" is invalid:
	Value {"allowSeparatedGroups":true,"ignoreCase":true,"ignoreDeclarationSort":false,"ignoreMemberSort":false} should NOT have additional properties.

Only to search around a bit and find this:

I love it when things you want show up just in time.

For those wondering, this landed in version 7.5.0.

Cheers! 🙏