-
-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add storm-like helpers #151
Conversation
return list | ||
} | ||
|
||
func (ol *OptionsList) Remove(key string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported method OptionsList.Remove should have comment or be unexported
@@ -519,10 +525,10 @@ func New() *Config { | |||
return &config | |||
} | |||
|
|||
// Open returns a Config object loaded with standard configuration file paths | |||
func Open() (*Config, error) { | |||
// OpenPath parses a configuration file and returns a *Config object |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment on exported function Open should be of the form "Open ..."
Value string | ||
} | ||
|
||
// OptionList is a list of options |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment on exported type OptionsList should be of the form "OptionsList ..." (with optional leading article)
No description provided.