From f14f6c1673ea35e7dec637852e1a2b34a81d436b Mon Sep 17 00:00:00 2001
From: George Eaton <george@dxw.com>
Date: Wed, 9 Aug 2023 10:48:53 +0100
Subject: [PATCH] Tweak eslint-plugin-import groups

It looks like there's a bug[0] in the package which is ignoring our
attempts to group `builtin` and `external` imports introduced with the
update to `2.28.0`.

This groups internal, parent and sibling imports together and seems to
fix the issue for now.

[0]: https://github.com/import-js/eslint-plugin-import/blob/1fa29717518732e3143aaf8155713133c383bf3c/src/rules/order.js#L431

Co-authored-by: Ynda Jas<ynda@dxw.com>
---
 .eslintrc.json | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/.eslintrc.json b/.eslintrc.json
index ec3372b74..df4600a41 100644
--- a/.eslintrc.json
+++ b/.eslintrc.json
@@ -71,7 +71,10 @@
       "error",
       {
         "alphabetize": { "order": "asc", "orderImportKind": "asc" },
-        "groups": [["builtin", "external"]],
+        "groups": [
+          ["builtin", "external"],
+          ["internal", "parent", "sibling"]
+        ],
         "newlines-between": "always"
       }
     ],