Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

Review the rationale and naming of the validate methods in options.go #341

Closed
alcortesm opened this issue Apr 12, 2017 · 2 comments
Closed

Review the rationale and naming of the validate methods in options.go #341

alcortesm opened this issue Apr 12, 2017 · 2 comments
Labels
enhancement

Comments

@alcortesm
Copy link
Contributor

see comments about this in #178 and #339.
@smola
Copy link
Collaborator

smola commented Apr 12, 2017

@alcortesm Feel free to explain the problem and propose any alternative API.

@mcuadros
Copy link
Contributor

After search about this, looks like is not that weird having a Validate function returning an error. Also I can't image other alternative better than Validate

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@smola @mcuadros @alcortesm