Add strict mode option

Author: kyoshidajp

README.md

@@ -49,12 +49,12 @@ Usage:
   dep-doctor diagnose [flags]
 
 Flags:
-  -f, --file string      dependencies file path (default "Gemfile.lock")
+  -f, --file string      dependencies file path
   -h, --help             help for diagnose
   -i, --ignores string   ignore dependencies (separated by a space)
-  -p, --package string   package manager (default "bundler")
+  -p, --package string   package manager
+      --strict           exit with non-zero if warnings exist
   -y, --year int         max years of inactivity (default 5)
-
 ```
 
 For example:

cmd/diagnose.go

@@ -183,6 +183,7 @@ type Options struct {
 	filePath       string
 	ignores        string
 	year           int
+	strict         bool
 }
 
 func (o *Options) Ignores() []string {
@@ -242,7 +243,7 @@ var diagnoseCmd = &cobra.Command{
 		}
 
 		diagnoses := Diagnose(doctor, f, o.year, o.Ignores())
-		if err := Report(diagnoses); err != nil {
+		if err := Report(diagnoses, o.strict); err != nil {
 			os.Exit(1)
 		}
 	},
@@ -254,6 +255,7 @@ func init() {
 	diagnoseCmd.Flags().StringVarP(&o.filePath, "file", "f", "", "dependencies file path")
 	diagnoseCmd.Flags().StringVarP(&o.ignores, "ignores", "i", "", "ignore dependencies (separated by a space)")
 	diagnoseCmd.Flags().IntVarP(&o.year, "year", "y", MAX_YEAR_TO_BE_BLANK, "max years of inactivity")
+	diagnoseCmd.PersistentFlags().BoolVarP(&o.strict, "strict", "", false, "exit with non-zero if warnings exist")
 
 	if err := diagnoseCmd.MarkFlagRequired("package"); err != nil {
 		fmt.Println(err.Error())
@@ -263,7 +265,7 @@ func init() {
 	}
 }
 
-func Report(diagnoses map[string]Diagnosis) error {
+func Report(diagnoses map[string]Diagnosis, strict_mode bool) error {
 	errMessages, warnMessages, ignoredMessages := []string{}, []string{}, []string{}
 	errCount, warnCount, infoCount := 0, 0, 0
 	unDiagnosedCount, ignoredCount := 0, 0
@@ -319,7 +321,7 @@ func Report(diagnoses map[string]Diagnosis) error {
 		infoCount, ignoredCount),
 	)
 
-	if len(errMessages) > 0 {
+	if len(errMessages) > 0 || strict_mode && warnCount > 0 {
 		return errors.New("has error")
 	}
 

cmd/ruby/bundler/testdata/Gemfile_warning.lock

@@ -0,0 +1,20 @@
+GEM
+  remote: https://rubygems.org/
+  specs:
+    concurrent-ruby (1.1.5)
+    dotenv (2.7.2)
+    faker (1.9.3)
+      i18n (>= 0.7)
+    i18n (1.6.0)
+      concurrent-ruby (~> 1.0)
+    active_decorator-rspec (0.0.9)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  dotenv (~> 2.7)
+  faker (~> 1.9)
+
+BUNDLED WITH
+   1.17.2