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

Small correction in your Pakyow Example binder code. Reference for data ... #1

Merged
merged 1 commit into from Mar 23, 2012

Conversation

Projects
None yet
2 participants
Contributor

hby commented Mar 21, 2012

...being bound is 'bindable'. Not sure of your full intent here but I generally name binders with a 'Binder' suffix. So, this would be a 'UserBinder'. I didn't make that change here, though. This reads well, though. I'm looking forward to watching the progress.

Small correction in your Pakyow Example binder code. Reference for da…
…ta being bound is 'bindable'. Not sure of your full intent here but I generally name binders with a 'Binder' suffix. So, this would be a 'UserBinder'. I didn't make that change here, though. This reads well, though. I'm looking forward to watching the progress.

Thanks for this, since it was an example I didn't run the code so I didn't notice bindable missing. I'll add the Binder suffix to the class as well.

nclaburn added a commit that referenced this pull request Mar 23, 2012

Merge pull request #1 from hby/patch-1
Binder must use bindable variable.

@nclaburn nclaburn merged commit c79fa83 into nclaburn:master Mar 23, 2012

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