Added flags to avoid duplicate output on errors and print all forbidden licenses before exit (#60)

Co-authored-by: Yuan (Bob) Gong <4957653+Bobgy@users.noreply.github.com>

Author: lavrd

.gitignore

@@ -0,0 +1 @@
+.idea

README.md

@@ -120,7 +120,6 @@ Forbidden license type WTFPL for library github.com/logrusorgru/auroraexit statu
 This command analyzes a package's dependencies and determines if any are
 considered forbidden by the license classifer. See
 [github.com/google/licenseclassifier](https://github.com/google/licenseclassifier/blob/842c0d70d7027215932deb13801890992c9ba364/license_type.go#L323)
-
 for licenses considered forbidden.
 
 ## Usages

check.go

@@ -48,15 +48,25 @@ func checkMain(_ *cobra.Command, args []string) error {
 	if err != nil {
 		return err
 	}
+
+	// indicate that a forbidden license was found
+	found := false
+
 	for _, lib := range libs {
 		licenseName, licenseType, err := classifier.Identify(lib.LicensePath)
 		if err != nil {
 			return err
 		}
+
 		if licenseType == licenses.Forbidden {
 			fmt.Fprintf(os.Stderr, "Forbidden license type %s for library %v\n", licenseName, lib)
-			os.Exit(1)
+			found = true
 		}
 	}
+
+	if found {
+		os.Exit(1)
+	}
+
 	return nil
 }

main.go

@@ -70,6 +70,8 @@ func init() {
 func main() {
 	flag.Parse()
 	rootCmd.PersistentFlags().AddGoFlagSet(flag.CommandLine)
+	rootCmd.SilenceErrors = true // to avoid duplicate error output
+	rootCmd.SilenceUsage = true  // to avoid usage/help output on error
 
 	if err := rootCmd.Execute(); err != nil {
 		glog.Exit(err)