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

Reworked and cleaned up dev dependencies #553

Closed
wants to merge 8 commits into from
Closed

Conversation

kirs
Copy link
Collaborator

@kirs kirs commented Jun 14, 2014

Should be merged before 0.8.6 release.
/cc @arnvald

@kirs
Copy link
Collaborator Author

kirs commented Jun 14, 2014

Let's see if the Travis build will be success..

@kirs
Copy link
Collaborator Author

kirs commented Jun 14, 2014

So now we have an issue with undefined methodauthenticates_with_sorcery!' for TestUser:Class`. @arnvald, any ideas?

@arnvald
Copy link
Collaborator

arnvald commented Jun 15, 2014

The problem was order of gems in Gemfile (Sorcery was loaded before ORMs, that's why Adapters were not included in ORM classes, so this method wasn't added there). I've changed it and checked locally - it should work now, let's wait for Travis.

@arnvald
Copy link
Collaborator

arnvald commented Jun 15, 2014

Ugh, now it still fails in lots of cases, I have no idea why - Gemfile and Gemfile.rails4 are almost the same, but Mongoid passes on the former and fails on the latter.

@kirs kirs closed this Jun 15, 2014
@kirs kirs deleted the reworked-dev-deps branch June 15, 2014 18:18
@kirs
Copy link
Collaborator Author

kirs commented Jun 15, 2014

Merged manually.

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