Fix: nix-community#185 - Change rules for empty brackets

src/rules.rs

@@ -56,8 +56,8 @@ pub(crate) fn spacing() -> SpacingDsl {
         .test("[1 2 3]", "[ 1 2 3 ]")
         .inside(NODE_LIST).after(T!["["]).single_space_or_newline()
         .inside(NODE_LIST).before(T!["]"]).single_space_or_newline()
-        .test("[ ]", "[]")
-        .inside(NODE_LIST).between(T!["["], T!["]"]).no_space()
+        .test("[]", "[ ]")
+        .inside(NODE_LIST).between(T!["["], T!["]"]).single_space()
         .inside(NODE_LIST).between(VALUES, VALUES).single_space_or_newline()
         .inside(NODE_LIST).between(VALUES, TOKEN_COMMENT).single_space_or_optional_newline()
         .inside(NODE_LIST).between(TOKEN_COMMENT, VALUES).single_space_or_newline()
@@ -79,9 +79,9 @@ pub(crate) fn spacing() -> SpacingDsl {
         .inside(NODE_ATTR_SET).after(T!["{"]).single_space_or_newline()
         .inside(NODE_ATTR_SET).before(T!["{"]).when(next_parent_has_update).newline()
         .inside(NODE_ATTR_SET).before(T!["}"]).single_space_or_newline()
-        .inside(NODE_ATTR_SET).before(T!["}"]).when(next_parent_has_update).newline()      
-        .test("{ }", "{}")
-        .inside(NODE_ATTR_SET).between(T!["{"], T!["}"]).no_space()
+        .inside(NODE_ATTR_SET).before(T!["}"]).when(next_parent_has_update).newline()
+        .test("{}", "{ }")
+        .inside(NODE_ATTR_SET).between(T!["{"], T!["}"]).single_space()
         .inside(NODE_ATTR_SET).between(NODE_KEY_VALUE, NODE_KEY_VALUE).single_space_or_newline()
         .inside(NODE_ATTR_SET).between(NODE_KEY_VALUE, TOKEN_COMMENT).single_space_or_optional_newline()
 
@@ -397,7 +397,6 @@ pub(crate) fn indentation() -> IndentDsl {
                   92
                 ]
             "#)
-
         .rule("Indent parenthesized expressions")
             .inside(NODE_PAREN)
             .not_matching([T!["("],T![")"]])

test_data/indent_or_default.good.nix

@@ -1,4 +1,4 @@
 {
   catAttrs = builtins.catAttrs or
-    (attr: l: concatLists (map (s: if s ? ${attr} then [ s.${attr} ] else []) l));
+    (attr: l: concatLists (map (s: if s ? ${attr} then [ s.${attr} ] else [ ]) l));
 }

test_data/indented_lambda.good.nix

@@ -3,7 +3,7 @@
     {
       # parameter comment
       mkSectionName ? (name: libStr.escape [ "[" "]" ] name)
-    , mkKeyValue ? mkKeyValueDefault {} "="
+    , mkKeyValue ? mkKeyValueDefault { } "="
     }: attrsOfAttrs:
       mapAttrsToStringsSep "\n" mkSection attrsOfAttrs;
 }

test_data/issue-132.good.nix

@@ -1,5 +1,5 @@
 {
-  buildInputs = [] ++ stdenv.lib.optionals enableGui (with qt5; [ qtbase qtwebkit ])
+  buildInputs = [ ] ++ stdenv.lib.optionals enableGui (with qt5; [ qtbase qtwebkit ])
   ++ stdenv.lib.optionals enableJupyter [ boost jsoncpp openssl zmqpp ]
   ;
 }

test_data/issue-185.bad.nix

@@ -0,0 +1,4 @@
+{
+attrs = {};
+lists = [ ];
+}

test_data/issue-185.good.nix

@@ -0,0 +1,4 @@
+{
+  attrs = { };
+  lists = [ ];
+}

test_data/list_elements.good.nix

@@ -1,6 +1,6 @@
 [
   1
-  []
+  [ ]
   { a = 92; }
   "hello"
   foo.bar

test_data/nested_indent.good.nix

@@ -1,7 +1,7 @@
-{ pkgs ? import ./nix/nixpkgs.nix {}
+{ pkgs ? import ./nix/nixpkgs.nix { }
 , src ? builtins.fetchGit {
     url = ./.;
     ref = "HEAD";
   }
 }:
-pkgs.example rec {}
+pkgs.example rec { }

test_data/syntax_errors/issue-93.bad.nix

@@ -1 +1 @@
-{ } =
+{} =

test_data/syntax_errors/issue-93.good.nix

@@ -1 +1 @@
-{} =
+{ } =

test_data/top_level_with2.good.nix

@@ -1,4 +1,4 @@
-with import <nixpkgs> {};
+with import <nixpkgs> { };
 stdenv.mkDerivation {
   name = "rnix";
   buildInputs = [ cargo ];