fix(config): qri config set on an int field does not actually update

qri-io · Apr 12, 2018 · 435c634 · 435c634
1 parent dba0b14
commit 435c634
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/cmd/cmd_test.go b/cmd/cmd_test.go
@@ -143,6 +143,7 @@ func TestCommandsIntegration(t *testing.T) {
  {"setup", "--peername=" + "alan"},
  {"config", "get"},
  {"config", "get", "profile"},
+ {"config", "set", "webapp.port", "3505"},
  // TODO - add setting whole config via a file
  // {"config", "set", "-i" + profileDataFilepath},
  {"info", "me"},

diff --git a/config/config.go b/config/config.go
@@ -113,6 +113,8 @@ func (cfg *Config) Set(path string, value interface{}) error {
  }
 
  switch v.Kind() {
+ case reflect.Int:
+ v.SetInt(rv.Int())
  case reflect.String:
  v.SetString(rv.String())
  case reflect.Bool: