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.

Sign up for GitHub

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

Jump to bottom

Add field mixin to type configurations ? #97

Closed

xuorig wants to merge 1 commit into rmosolgo:master from xuorig:mixins

Contributor

xuorig commented Feb 16, 2016

Small PR just to spark discussion of adding a mixin method in DefinedByConfig like proposed by @rmosolgo in #91

Mixin class is simply a DefinedByConfig class containing fields only for now.


          Add field mixin to type configurations

7ee571d

xuorig changed the title ~~Add field mixin to type configurations~~ Add field mixin to type configurations ?

Owner

rmosolgo commented Feb 19, 2016

For my part, I was convinced by: #91 (comment)

It will make field lookups a bit tougher, but it will make interfaces actually useful!

xuorig closed this

xuorig deleted the mixins branch

February 29, 2016 20:11

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

Labels

None yet