Skip to content

Conversation

@jswank
Copy link

@jswank jswank commented May 22, 2012

Renamed Config.data to Config.Data and merge() to Merge().

I found these minor changes to be convenient wrt/ setting defaults for optional configuration directives or overriding config directives based on other input.

    defaults := map[string]interface{}{
        "foo": "on",
        "bar": 1,
        "baz": true,
    }

    config := &jconfig.Config{}
    config.Data = defaults

    config.StringMerge(`{"foo": "off"}`)

    fmt.Printf("optional config baz: %t\n", config.GetBool("baz"))

export merge() function (rename merge() to Merge())
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant