[FIX] CWL: positional options are required

positional list options are optional

Author: eseiler

include/sharg/detail/format_tdl.hpp

@@ -298,12 +298,14 @@ class format_tdl : format_base
                 ++positional_option_count;
                 auto description = config.description + default_message + validator_message;
 
-                parameters.push_back(tdl::Node{
-                    .name = id,
-                    .description = description,
-                    .tags = {},
-                    .value = tdl::StringValue{},
-                });
+                parameters.push_back(tdl::Node{.name = id,
+                                               .description = description,
+                                               .tags = {detail::is_container_option<option_type> ? "" : "required"}});
+
+                if constexpr (detail::is_container_option<option_type>)
+                    parameters.back().value = tdl::StringValueList{};
+                else
+                    parameters.back().value = tdl::StringValue{};
             });
     }
 

test/unit/detail/format_cwl_test.cpp

@@ -85,10 +85,10 @@ TEST(format_cwl_test, full_information)
                                  "inputs:\n"
                                  "  positional_0:\n"
                                  "    doc: this is a positional option.\n"
-                                 "    type: string?\n"
+                                 "    type: string\n"
                                  "  positional_1:\n"
                                  "    doc: \"this is a positional option. Default: []\"\n"
-                                 "    type: string?\n"
+                                 "    type: string[]?\n"
                                  "  int:\n"
                                  "    doc: \"this is a int option. Default: 5\"\n"
                                  "    type: long?\n"
@@ -215,10 +215,10 @@ TEST(format_cwl_test, subparser)
         "inputs:\n"
         "  positional_0:\n"
         "    doc: this is a positional option.\n"
-        "    type: string?\n"
+        "    type: string\n"
         "  positional_1:\n"
         "    doc: \"this is a positional option. Default: []\"\n"
-        "    type: string?\n"
+        "    type: string[]?\n"
         "  int:\n"
         "    doc: \"this is a int option. Default: 5\"\n"
         "    type: long?\n"