Skip to content

Commit

Permalink
PR(ADHERE-REVIEW): Use StringVar for only long flag arg
Browse files Browse the repository at this point in the history
  • Loading branch information
shahzadlone committed Oct 4, 2024
1 parent 725c674 commit b48a4e3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions cli/acp_relationship_add.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,11 +117,10 @@ Example: Creating a dummy relationship does nothing (from database prespective):
)
_ = cmd.MarkFlagRequired(targetActorFlagLong)

cmd.Flags().StringVarP(
cmd.Flags().StringVar(
&docIDArg,
docIDFlag,
"",
"",
"Document Identifier (ObjectID) to make relationship for",
)
_ = cmd.MarkFlagRequired(docIDFlag)
Expand Down
3 changes: 1 addition & 2 deletions cli/acp_relationship_delete.go
Original file line number Diff line number Diff line change
Expand Up @@ -109,11 +109,10 @@ Example: Let another actor (4d092126012ebaf56161716018a71630d99443d9d5217e9d8502
)
_ = cmd.MarkFlagRequired(targetActorFlagLong)

cmd.Flags().StringVarP(
cmd.Flags().StringVar(
&docIDArg,
docIDFlag,
"",
"",
"Document Identifier (ObjectID) to delete relationship for",
)
_ = cmd.MarkFlagRequired(docIDFlag)
Expand Down

0 comments on commit b48a4e3

Please sign in to comment.