Skip to content
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

highlight the need to require the Feature model somewhere first #7

Merged
merged 1 commit into from
Jul 1, 2013
Merged

highlight the need to require the Feature model somewhere first #7

merged 1 commit into from
Jul 1, 2013

Conversation

kcm
Copy link
Contributor

@kcm kcm commented Jun 5, 2013

Per #1

pda added a commit that referenced this pull request Jul 1, 2013
@pda
Merge pull request #7 from kcm/feature/readme-model
23fc1f5 
highlight the need to require the Feature model somewhere first
@pda pda merged commit 23fc1f5 into pda:master Jul 1, 2013
@pda
Copy link
Owner

pda commented Jul 1, 2013

Sorry for letting this hang for ages.
The long-term solution will be changing flip to declare the features in an initializer, not in the persistence model class. But this is helpful advice in the mean time :)

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

Successfully merging this pull request may close these issues.
2 participants
@kcm @pda