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.

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

Fix initialization error when inheriting #83

Closed
wants to merge 1 commit into from

Conversation

pwim
Copy link
Collaborator

@pwim pwim commented Sep 7, 2017

Depending on loading order, @mobility may have not been initialized yet, so use method to ensure we aren't trying to dup nil.

Resolves #82.

Depending on loading order, @mobility may have not been initialized yet,
so use method to ensure we aren't trying to dup nil.
@shioyama
Copy link
Owner

shioyama commented Sep 7, 2017

Thanks! Will merge once specs pass.

@shioyama
Copy link
Owner

shioyama commented Sep 7, 2017

The failure is because Sequel 5 was released and there's a compatibility issue. I'll peg the version and get the remaining specs to pass.

@shioyama
Copy link
Owner

shioyama commented Sep 7, 2017

Cherry-picked this in cf3860f. Thanks!

@shioyama shioyama closed this Sep 7, 2017
@shioyama
Copy link
Owner

Just FYI: I released 0.2.3 which only has this fix, and created a new branch 0-2-stable: to track any further minor bug fixes prior to releasing 0.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants