env-set command now sends multiple variables at once #153

Merged
merged 3 commits into from Sep 25, 2015

Projects

None yet

2 participants

@Zyko0
Contributor
Zyko0 commented Sep 25, 2015

No description provided.

@Soulou Soulou and 1 other commented on an outdated diff Sep 25, 2015
- if code == 201 {
- fmt.Printf("%s has been set to %s.\n", name, value)
- } else if code == 200 {
- fmt.Printf("%s has been updated to %s.\n", name, value)
- }
+ _, _, err := scalingo.VariableMultipleSet(app, variables)
+ if err != nil {
+ return errgo.Mask(err, errgo.Any)
+ }
+
+ for _, variable := range variables {
+ fmt.Printf("%s set to %s.\n", variable.Name, variable.Value)
@Soulou
Soulou Sep 25, 2015 Member

Please respect previous wording. 'set' -> 'has been set'

@Zyko0
Zyko0 Sep 25, 2015 Contributor

Sorry about that, I didn't remember the exact wording

@Soulou
Member
Soulou commented Sep 25, 2015

You didn't run godep save, dependencies are out of date.

@Soulou Soulou merged commit 8c63dea into Scalingo:master Sep 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment