Skip to content

Commit 1e9c53e

Browse files
authored
docs: update suggestions to use "set" and "unset" environment commands (#525)
1 parent 1570564 commit 1e9c53e

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

.github/STYLE_GUIDE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ The square brackets surrounding command arguments hint that these are optional:
3131

3232
```
3333
USAGE
34-
$ slack env add [name] [value] [flags]
34+
$ slack env set [name] [value] [flags]
3535
```
3636

3737
The angled brackets around arguments hint that these are required:

cmd/env/unset.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ func runEnvUnsetCommandFunc(clients *shared.ClientFactory, cmd *cobra.Command, a
112112
Emoji: "evergreen_tree",
113113
Text: "Environment Unset",
114114
Secondary: []string{
115-
"The app has no environment variables to remove",
115+
"The app has no environment variables to unset",
116116
},
117117
}))
118118
return nil
@@ -141,7 +141,7 @@ func runEnvUnsetCommandFunc(clients *shared.ClientFactory, cmd *cobra.Command, a
141141
Emoji: "evergreen_tree",
142142
Text: "Environment Unset",
143143
Secondary: []string{
144-
"The project has no environment variables to remove",
144+
"The project has no environment variables to unset",
145145
},
146146
}))
147147
return nil

cmd/env/unset_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ func Test_Env_RemoveCommand(t *testing.T) {
7979
cm.API.AssertNotCalled(t, "RemoveVariable")
8080
},
8181
ExpectedStdoutOutputs: []string{
82-
"The project has no environment variables to remove",
82+
"The project has no environment variables to unset",
8383
},
8484
},
8585
"exit without errors when hosted app has zero variables": {
@@ -94,7 +94,7 @@ func Test_Env_RemoveCommand(t *testing.T) {
9494
cm.API.AssertNotCalled(t, "RemoveVariable")
9595
},
9696
ExpectedStdoutOutputs: []string{
97-
"The app has no environment variables to remove",
97+
"The app has no environment variables to unset",
9898
},
9999
},
100100
"remove a hosted variable using arguments": {

internal/auth/auth.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -494,7 +494,7 @@ func (c *Client) ResolveAPIHost(ctx context.Context, apiHostFlag string, customA
494494
c.io.PrintDebug(
495495
ctx,
496496
"You're using a custom apihost. Run %s to add it to your app's Run on Slack environment",
497-
style.Commandf(fmt.Sprintf("var add SLACK_API_URL %s", apiHost), false),
497+
style.Commandf(fmt.Sprintf("env set SLACK_API_URL %s", apiHost), false),
498498
)
499499
}
500500

0 commit comments

Comments
 (0)