fix a few comments, and change default version flag to only use "--version"

Author: phlptp

examples/CMakeLists.txt

@@ -54,7 +54,7 @@ set_property(TEST simple_all PROPERTY PASS_REGULAR_EXPRESSION
     "Received flag: 2 (2) times"
     "Some value: 1.2")
 
-add_test(NAME simple_version COMMAND simple -v)
+add_test(NAME simple_version COMMAND simple --version)
 set_property(TEST simple_version PROPERTY PASS_REGULAR_EXPRESSION
     "${CLI11_VERSION}")
 

include/CLI/App.hpp

@@ -746,13 +746,13 @@ class App {
     }
     /// Set the version as a string
     App *version(const std::string &versionString) {
-        set_version_flag("-v,--version", versionString);
+        set_version_flag("--version", versionString);
         return this;
     }
 
     /// Generate the version string through a callback function
     App *version(std::function<std::string()> vfunc) {
-        set_version_flag("-v,--version", vfunc);
+        set_version_flag("--version", vfunc);
         return this;
     }
 

include/CLI/StringTools.hpp

@@ -190,7 +190,7 @@ inline bool valid_name_string(const std::string &str) {
     return true;
 }
 
-/// check if a string is a container segment separator (empty or "%%"
+/// check if a string is a container segment separator (empty or "%%")
 inline bool is_separator(const std::string &str) {
     static const std::string sep("%%");
     return (str.empty() || str == sep);

include/CLI/TypeTools.hpp

@@ -232,7 +232,7 @@ struct is_mutable_container<
 // check to see if an object is a mutable container (fail by default)
 template <typename T, typename _ = void> struct is_readable_container : std::false_type {};
 
-/// type trait to test if a type is a container meaning it has a value_type, it has an iterator, a clear, and an en end
+/// type trait to test if a type is a container meaning it has a value_type, it has an iterator, a clear, and an end
 /// methods and an insert function.  And for our purposes we exclude std::string and types that can be constructed from
 /// a std::string
 template <typename T>
@@ -244,7 +244,7 @@ struct is_readable_container<
 // check to see if an object is a wrapper (fail by default)
 template <typename T, typename _ = void> struct is_wrapper : std::false_type {};
 
-// check if an object is a is a wrapper (it has a value_type defined)
+// check if an object is a wrapper (it has a value_type defined)
 template <typename T>
 struct is_wrapper<T, conditional_t<false, void_t<typename T::value_type>, void>> : public std::true_type {};
 
@@ -344,7 +344,7 @@ auto value_string(const T &value) -> decltype(to_string(value)) {
     return to_string(value);
 }
 
-/// temple to get the underlying value type if it exists or use a default
+/// template to get the underlying value type if it exists or use a default
 template <typename T, typename def, typename Enable = void> struct wrapped_type { using type = def; };
 
 /// Type size for regular object types that do not look like a tuple

tests/HelpTest.cpp

@@ -1220,7 +1220,7 @@ TEST(TVersion, parse_throw) {
     app.version(CLI11_VERSION);
 
     EXPECT_THROW(app.parse("--version"), CLI::CallForVersion);
-    EXPECT_THROW(app.parse("-v --arg2 5"), CLI::CallForVersion);
+    EXPECT_THROW(app.parse("--version --arg2 5"), CLI::CallForVersion);
 
     auto ptr = app.get_version_ptr();