Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Support arbitrary option types #35

Merged
merged 2 commits into from May 22, 2012

Conversation

Projects
None yet
2 participants
Contributor

oscarbenjamin commented May 22, 2012

This is a better patch for #33

@piranha piranha commented on an outdated diff May 22, 2012

@@ -470,11 +470,15 @@ def default_value(self):
class LiteralOption(BaseOption):
- '''Literal option type (including string options, no processing).'''
- type = object
+ '''Literal option type (including string, int, float, etc.)'''
+ type=object
@piranha

piranha May 22, 2012

Owner

Ehm, can we get spaces around = here please? :)

piranha pushed a commit that referenced this pull request May 22, 2012

Merge pull request #35 from oscarbenjamin/newopts
Support arbitrary option types

@piranha piranha merged commit cff75fe into piranha:master May 22, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment