Skip to content

Commit

Permalink
chore(cli): rename Redis flags for consistency and clarity
Browse files Browse the repository at this point in the history
The flags for Redis service host and port are renamed from
`redis-master-service-host` and `redis-master-service-port` to
`redis-service-host` and `redis-service-port`. This change improves
consistency in naming and clarifies that these flags refer to the
Redis service in general, rather than a specific master instance.
  • Loading branch information
cybersiddhu committed Oct 15, 2024
1 parent b0cdd89 commit 3ffbf96
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions internal/uniprot/cli/flag.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,17 +21,17 @@ func UniprotFlags() []cli.Flag {
Value: uniprotURL,
},
&cli.StringFlag{
Name: "redis-master-service-host",
Name: "redis-service-host",
Aliases: []string{"s"},
Usage: "Redis service host address",
EnvVars: []string{"REDIS_MASTER_SERVICE_HOST"},
EnvVars: []string{"REDIS_SERVICE_HOST"},
},
&cli.StringFlag{
Name: "redis-master-service-port",
Name: "redis-service-port",
Aliases: []string{"p"},
Usage: "Redis service port",
Value: "6379",
EnvVars: []string{"REDIS_MASTER_SERVICE_PORT"},
EnvVars: []string{"REDIS_SERVICE_PORT"},
},
}
}
4 changes: 2 additions & 2 deletions internal/uniprot/client/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ func SetRedisClient(cltx *cli.Context) error {
client := rds.NewClient(&rds.Options{
Addr: fmt.Sprintf(
"%s:%s",
cltx.String("redis-master-service-host"),
cltx.String("redis-master-service-port"),
cltx.String("redis-service-host"),
cltx.String("redis-service-port"),
),
})
ctx := context.Background()
Expand Down

0 comments on commit 3ffbf96

Please sign in to comment.