Merge pull request #175 from mediamonks/hexadecimal-number-formatting

Make grouping optional for all number types except `number`
mediamonks · Oct 20, 2023 · 69b1c19 · 69b1c19
2 parents 05e0480 + 54001b0
commit 69b1c19
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/packages/eslint-config/index.js b/packages/eslint-config/index.js
@@ -272,6 +272,15 @@ module.exports = {
     'unicorn/prefer-string-replace-all': 'error',
     'unicorn/require-post-message-target-origin': 'error',
     // 'unicorn/string-content': 'error',
+    'unicorn/numeric-separators-style': [
+      'error',
+      {
+        onlyIfContainsSeparator: true,
+        number: {
+          onlyIfContainsSeparator: false,
+        },
+      },
+    ],
 
     /**
      * Import

diff --git a/packages/test-eslint-config/numericSeperatorStyle.js b/packages/test-eslint-config/numericSeperatorStyle.js
@@ -0,0 +1,11 @@
+/* eslint-disable no-unused-vars */
+
+const hex1 = 0xff0000;
+const hex2 = 0xff_00_00;
+const number1 = 100;
+const number2 = 1000;
+const number3 = 1_000_000;
+const number4 = 1.234_56;
+const octal1 = 0o7777;
+const octal3 = 0o12_7777;
+const binary = 0b101010001;