Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[#38] updated sumo_db dep and now it is working with OTP-19 #39

Merged
merged 2 commits into from
Sep 6, 2016

Conversation

ferigis
Copy link
Member

@ferigis ferigis commented Sep 6, 2016

Upgraded sumo_db in order to make sumo_rest OTP-19 compatible. [close #38 ]

@@ -52,10 +53,10 @@ sumo_schema() ->
, sumo:new_field(updated_at, datetime, [not_null])
]).

-spec sumo_sleep(element()) -> sumo:doc().
-spec sumo_sleep(sumo:user_doc()) -> sumo:model().
Copy link
Member

@elbrujohalcon elbrujohalcon Sep 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be…

-spec sumo_sleep(element()) -> sumo:model().

What's the error reported by dialyzer?

@elbrujohalcon elbrujohalcon merged commit 3566255 into master Sep 6, 2016
@ferigis ferigis deleted the ferigis.38.upgrade_deps branch September 14, 2016 11:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade dependencies
2 participants