chore: Polished CLI output a bit

[rhuss]

• Added SilenceUsage: true to the command configuration to suppress usage messages.
• Simplified error handling in the GitHub client by removing unnecessary buffer variable.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rhuss
Once this PR has been reviewed and has the lgtm label, please assign cardil for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[cardil]

Why are you removing the Print entirely?

I can agree to remove it if err == nil. Otherwise, the GitHub Client output might contain important debugging info.

[rhuss]

Actually for me the output was just an array of ints (that can be decoded to some URLs). It was not even useful in the error case.

[rhuss]

but we can keep it for the error case, sure.

[cardil]

Good point. It's a []byte not string.

It should be decoded before printing. Similarly, like here:

deviate/pkg/sync/create_pr.go

Lines 69 to 81 in ead21be

	if err != nil {
	return nil, errors.Wrap(err, ErrSyncFailed)
	}
	un := make([]map[string]interface{}, 0)
	err = json.Unmarshal(buff, &un)
	if err != nil {
	return nil, errors.Wrap(err, ErrSyncFailed)
	}
	if len(un) > 0 {
	u := fmt.Sprintf("%s", un[0]["url"])
	return &u, nil
	}