Set indent-with-indicator before indicator-indent in dumper options (#…

…142)
clj-commons · Nov 22, 2024 · f01e8f8 · f01e8f8
1 parent ca228f2
commit f01e8f8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/src/clojure/clj_yaml/core.clj b/src/clojure/clj_yaml/core.clj
@@ -72,10 +72,10 @@
       (.setDefaultFlowStyle dumper (flow-styles flow-style)))
     (when indent
       (.setIndent dumper indent))
-    (when indicator-indent
-      (.setIndicatorIndent dumper indicator-indent))
     (when indent-with-indicator
       (.setIndentWithIndicator dumper indent-with-indicator))
+    (when indicator-indent
+      (.setIndicatorIndent dumper indicator-indent))
     dumper))
 
 (defn default-loader-options

diff --git a/test/clj_yaml/core_test.clj b/test/clj_yaml/core_test.clj
@@ -411,7 +411,20 @@ lol: yolo")
                             :dumper-options {:indent 5
                                              :indicator-indent 2
                                              :indent-with-indicator true
-                                             :flow-style :block})))))
+                                             :flow-style :block})))
+    (is (= (str "todo:\n"
+                ;12
+                "  issues:\n"
+                ;; 12
+                "    - name: Fix all the things\n"
+                "      responsible:\n"
+                ;;     12
+                "        name: Rita\n")
+           (generate-string (parse-string indent-yaml)
+             :dumper-options {:indent 2
+                              :indicator-indent 2
+                              :indent-with-indicator true
+                              :flow-style :block})))))
 
 (def yaml-with-unknown-tags "---
 scalar: !CustomScalar some-scalar