Fix version information

README.md

@@ -401,10 +401,8 @@ This scenario handles two problems:
 | `-version`      | `bool`   | Show version of application                          |
 | `-help`         | `bool`   | Show usage and flags                                 |
 
-Boolean flags (e.g. `-fileAppend` and `-help`) do not require a value.
+Boolean flags (e.g. `-version` and `-help`) do not require a value.
 The presence of such a flag indicates a value of `true`.
-In the case of `-version`, which defaults to `true`,
-it is necessary to specify `-version=false` to turn it off.
 
 Format values can be set separately for console output and optional log file.
 

tester/flags/flags.go

@@ -49,7 +49,7 @@ func NewSet() *Set {
 	set.StringVar(&set.logFileFormat, "fileFormat", logging.FmtDefault, "Log file format")
 	set.UintVar(&set.maxFieldLen, "maxFieldLen", 32, "Maximum length of fields to display")
 	set.BoolVar(&set.logMsgTwice, "logMsgTwice", false, "Log each message twice with tester in the middle")
-	set.BoolVar(&set.version, "version", true, "Show lsp-tester version")
+	set.BoolVar(&set.version, "version", false, "Show lsp-tester version")
 	return set
 }
 

tester/main.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"fmt"
 	"net"
 	"os"
 	"runtime/debug"
@@ -112,9 +111,9 @@ func logVersion() {
 			case "vcs.revision":
 				event.Str("Revision", setting.Value)
 			case "vcs.time":
-				event.Str("When", setting.Value)
+				event.Str("Revised", setting.Value)
 			case "vcs.modified":
-				event.Str("Dirty", setting.Value)
+				event.Str("Modified", setting.Value)
 			}
 		}
 		if target != "" && arch != "" {
@@ -125,7 +124,5 @@ func logVersion() {
 		if target != "" {
 			event.Str("Target", target).Msg("Version")
 		}
-
-		fmt.Print(info)
 	}
 }