Merge pull request #393 from jhkimqd/jihwan/monitor-silence-usage

feat: silence usage when error returned for monitor
0xPolygon · Oct 7, 2024 · 989b4a2 · 989b4a2
2 parents d97cc5b + 86f5fb7
commit 989b4a2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/cmd/monitor/cmd.go b/cmd/monitor/cmd.go
@@ -51,10 +51,11 @@ func (s *SafeBatchSize) Auto() bool {
 }
 
 var MonitorCmd = &cobra.Command{
-	Use:   "monitor",
-	Short: "Monitor blocks using a JSON-RPC endpoint.",
-	Long:  usage,
-	Args:  cobra.NoArgs,
+	Use:          "monitor",
+	Short:        "Monitor blocks using a JSON-RPC endpoint.",
+	Long:         usage,
+	Args:         cobra.NoArgs,
+	SilenceUsage: true,
 	PersistentPreRunE: func(cmd *cobra.Command, args []string) error {
 		// By default, hide logs from `polycli monitor`.
 		verbosityFlag := cmd.Flag("verbosity")

diff --git a/cmd/monitor/monitor.go b/cmd/monitor/monitor.go
@@ -167,7 +167,7 @@ func monitor(ctx context.Context) error {
 			for {
 				err = fetchCurrentBlockData(ctx, ec, ms, isUiRendered)
 				if err != nil {
-					log.Error().Msg(fmt.Sprintf("Error: %v", err))
+					log.Error().Msg(fmt.Sprintf("Error: unable to fetch current block data: %v", err))
 					// Send the error to the errChan channel to return.
 					errChan <- err
 					return